Skip to content

One of compound scope type#1069

Merged
pokey merged 7 commits intomainfrom
one_of_compound_scope_type
Oct 24, 2022
Merged

One of compound scope type#1069
pokey merged 7 commits intomainfrom
one_of_compound_scope_type

Conversation

@AndreasArvidsson
Copy link
Member

Fixes #1045

Checklist

@AndreasArvidsson AndreasArvidsson marked this pull request as draft October 23, 2022 10:55
@AndreasArvidsson
Copy link
Member Author

@pokey Added this draft so we can discuss it on the meet up

@pokey pokey force-pushed the one_of_compound_scope_type branch from 5baeecf to fb5d268 Compare October 23, 2022 16:14
@pokey pokey changed the base branch from main to pokey/simplify-default-iteration-scopes-file-scopehandler October 23, 2022 16:15
Copy link
Member

@pokey pokey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok I rebased onto #1075 and tweaked your implementation. Lmk what you think

Not totally sold on the name oneOf; let's discuss at meet-up

Base automatically changed from pokey/simplify-default-iteration-scopes-file-scopehandler to main October 24, 2022 08:22
@pokey pokey force-pushed the one_of_compound_scope_type branch from 68543ea to f5379b1 Compare October 24, 2022 10:29
pokey
pokey previously approved these changes Oct 24, 2022
@pokey pokey marked this pull request as ready for review October 24, 2022 19:26
Copy link
Member

@pokey pokey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess let's merge this in for now

@pokey pokey merged commit f1e7b4c into main Oct 24, 2022
@pokey pokey deleted the one_of_compound_scope_type branch October 24, 2022 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add new oneOf compound scope type

2 participants