Skip to content
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

Fix CI jobs #3763

Merged
merged 17 commits into from
Oct 8, 2024
Merged

Fix CI jobs #3763

merged 17 commits into from
Oct 8, 2024

Conversation

jakemac53
Copy link
Contributor

@jakemac53 jakemac53 commented Oct 8, 2024

This is an absolute mess now, our workspace is super fragmented at this point, but in general this:

  • Fixes new analyzer deprecations
  • Requires the new analyzer
    • Which requires a new SDK
    • Which requires moving more packages out of the workspace
  • Fixes the jobs that run against main by allowing 3.7.0 dev SDKs. These versions will be broken at some point because the DDC binary is going to move, so we are not allowing 3.7.0 stable.

@jakemac53
Copy link
Contributor Author

cc @scheglov do you know what min analyzer version I need for these new apis (if any)?

Copy link

github-actions bot commented Oct 8, 2024

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

Package publish validation ✔️
Package Version Status
package:build 2.4.2-wip WIP (no publish necessary)
package:build_config 1.1.2-wip WIP (no publish necessary)
package:build_daemon 4.0.3-wip WIP (no publish necessary)
package:build_modules 5.0.10-wip WIP (no publish necessary)
package:build_resolvers 2.4.3-wip WIP (no publish necessary)
package:build_runner 2.4.13 already published at pub.dev
package:build_runner_core 7.3.3-wip WIP (no publish necessary)
package:build_test 2.2.3-wip WIP (no publish necessary)
package:build_web_compilers 4.1.0-wip WIP (no publish necessary)
package:scratch_space 1.0.3-wip WIP (no publish necessary)

Documentation at https:/dart-lang/ecosystem/wiki/Publishing-automation.

@scheglov
Copy link
Contributor

scheglov commented Oct 8, 2024

cc @scheglov do you know what min analyzer version I need for these new apis (if any)?

I think 6.9.0 should be enough, older than that not sure.

@jakemac53 jakemac53 mentioned this pull request Oct 8, 2024
@jakemac53 jakemac53 changed the title fix analyzer deprecations Fix CI jobs Oct 8, 2024
@jakemac53 jakemac53 merged commit c9a64a4 into master Oct 8, 2024
79 checks passed
@jakemac53 jakemac53 deleted the fix-deprecations branch October 8, 2024 21:11
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.

2 participants