Skip to content
This repository has been archived by the owner on Nov 20, 2024. It is now read-only.

enable combinators_ordering #3524

Merged
merged 1 commit into from
Jul 19, 2022
Merged

enable combinators_ordering #3524

merged 1 commit into from
Jul 19, 2022

Conversation

pq
Copy link
Member

@pq pq commented Jul 13, 2022

(And remove deprecated invariant_booleans.)

/cc @bwilkerson

/fyi @a14n

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.682% when pulling 0b0f549 on enable_combinators_ordering into ee4e627 on main.

@pq
Copy link
Member Author

pq commented Jul 13, 2022

Ah. It looks like combinators_ordering isn't in a dev build yet. We'll have to wait on this until it is.

@bwilkerson
Copy link
Member

Do we have a fix for this? We really ought to. It's mechanical and tedious to fix by hand.

@pq
Copy link
Member Author

pq commented Jul 13, 2022

Do we have a fix for this? We really ought to. It's mechanical and tedious to fix by hand.

We do not.

/fyi @jcollins-g

@pq pq merged commit 6e91a47 into main Jul 19, 2022
@pq pq deleted the enable_combinators_ordering branch July 19, 2022 12:51
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Aug 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants