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

Widen Dependency Ranges Blocking Dart 2.13 #352

Merged

Conversation

sourcegraph-wk
Copy link

Motivation

In order for projects to be compatible with Dart 2.13, there are dependencies that need to have their ranges widened.
For a list of dependencies expected to be changed, see the dependencies section of the Dart 2.12+ wiki page.

Another necessary change is the removal of build_vm_compilers. If this is the case, additional clean up may need to be done, such as removing the associated config from build.yaml or updating how tests are run. A member of Client Platform will be following up with PRs that have CI failures to resolve any issues or perform necessary clean up.

For any questions or concerns, don't hesitate to reach us in the #support-client-plat Slack channel!

Changes

  • Update specific dependencies to be compatible with Dart 2.13.
  • Remove build_vm_compilers if it was present

QA

  • CI passes

Created by Sourcegraph batch change Workiva/dart_213_dependency_range_widen.

@aviary-wf
Copy link

Security Insights

No security relevant content was detected by automated scans.

Action Items

  • Review PR for security impact; comment "security review required" if needed or unsure
  • Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

@aaronlademann-wf aaronlademann-wf force-pushed the campaign/client_platform/dart-213-dependency-range-widen branch from 271b428 to 4f95d7d Compare July 21, 2021 16:55
@aaronlademann-wf aaronlademann-wf force-pushed the campaign/client_platform/dart-213-dependency-range-widen branch from a1fd38f to f0c20dd Compare July 21, 2021 17:01
@evanweible-wf
Copy link
Contributor

QA +1

  • CI passes on 2.7.2, stable, and dev

@Workiva/release-management-p

Copy link
Contributor

@rmconsole-wf rmconsole-wf left a comment

Choose a reason for hiding this comment

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

+1 from RM

@rmconsole5-wk rmconsole5-wk merged commit 5aa115e into master Jul 21, 2021
@rmconsole5-wk rmconsole5-wk deleted the campaign/client_platform/dart-213-dependency-range-widen branch July 21, 2021 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants