Skip to content
Snippets Groups Projects

Rename `option_union_Some` → `union_Some`

Merged Robbert Krebbers requested to merge robbert/union_Some into master

See discussion at !432 (comment 87085)

This removes option_ if there's already something else in the name to disambiguate, here that is _Some.

We could also prefix everything with option_, but that would require more changes. For better or worse, I think this MR matches the consensus we also have for lists, maps, sets (e.g., having lookup_app_Some instead of list_lookup_app_Some).

Merge request reports

Merge request pipeline #79450 passed

Merge request pipeline passed for 7ff6b913

Approval is optional

Merged by Robbert KrebbersRobbert Krebbers 2 years ago (Mar 21, 2023 4:45pm UTC)

Merge details

  • Changes merged into with 20813c6c.
  • Deleted the source branch.

Pipeline #79451 passed

Pipeline passed for 20813c6c on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading