-
Notifications
You must be signed in to change notification settings - Fork 223
Allow analyzer major version 10. #2584
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
PR HealthChangelog Entry ✔️
Changes to files need to be accounted for in their respective changelogs. This check can be disabled by tagging the PR with |
|
Can we just do a release and skip the |
|
Will remove |
|
I don't understand why "Publish / publish / validate (pull_request)" is not happy. |
I think because |
|
The versions referenced in test_api: 0.7.9
test_core: 0.6.15But yes, they are not published yet, so maybe this is why it complains? I myself probably will not able to publish anyway... |
Package publishing
Documentation at https://github.com/dart-lang/ecosystem/wiki/Publishing-automation. |
Fwiw as soon as you bump these to released versions they should get published, but it is mostly automatic (click the links in the bot comment above - AFTER merging). We should wait for @natebosch though to comment on whether we are ready to publish there was some previous stuff getting validated internally. |
|
We're unblocked for publishing now, had some blockers earlier but we're synced to the SDK and google3 now so I would have opened a PR to publish this afternoon anyway. Thanks for the PR! |
No description provided.