Skip to content
This repository was archived by the owner on Jul 2, 2024. It is now read-only.

Remove deprecated authors field from pubspec.yaml#18

Merged
rmconsole2-wf merged 1 commit intomasterfrom
batch/client_platform/remove_pubspec_authors
Apr 26, 2021
Merged

Remove deprecated authors field from pubspec.yaml#18
rmconsole2-wf merged 1 commit intomasterfrom
batch/client_platform/remove_pubspec_authors

Conversation

@sourcegraph-wk
Copy link

This PR removes the author field from any pubspec.yaml files in this repo,
as the field was deprecated in Dart 2.7 and is no longer needed.

Removing this field will silence the warning that pub publish emits, which
has the added benefit of allowing us to use pub publish --dry-run as a
quality check during CI.

If you'd like to retain the author information, we recommend adding an
AUTHORS.md file in the repo or the package directory.


This PR was created automatically from a Sourcegraph batch change.
Please reach out to @evanweible-wf or #support-client-plat with any questions.

Created by Sourcegraph batch change Workiva/remove_pubspec_authors.

@aviary3-wk
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.

@sourcegraph-wk sourcegraph-wk force-pushed the batch/client_platform/remove_pubspec_authors branch from 25e537e to 521e449 Compare April 20, 2021 00:52
@robbecker-wf
Copy link
Member

@Workiva/release-management-pp

Copy link

@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

@rmconsole2-wf rmconsole2-wf merged commit e556b2a into master Apr 26, 2021
@rmconsole2-wf rmconsole2-wf deleted the batch/client_platform/remove_pubspec_authors branch April 26, 2021 15:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants