Skip to content

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.

@aviary2-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.

Since this repo has already migrated to null safety .. a minor adjustment to build_test was needed.
@robbecker-wf
Copy link
Member

@Workiva/release-management-p

@robbecker-wf
Copy link
Member

QA+1 CI passes

@robbecker-wf robbecker-wf merged commit 8b0ff3d into master Jul 21, 2021
@robbecker-wf robbecker-wf deleted the campaign/client_platform/dart-213-dependency-range-widen branch July 21, 2021 19:19
Copy link
Contributor

@joebingham-wk joebingham-wk left a comment

Choose a reason for hiding this comment

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

+1

@martinolson-wk
Copy link

RM +1

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