Skip to content

Conversation

@godofredoc
Copy link
Contributor

This will allow us to assess the security posture of this repository.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@godofredoc godofredoc requested a review from stevemessick March 18, 2022 22:30
Copy link
Member

@stevemessick stevemessick left a comment

Choose a reason for hiding this comment

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

Unit tests will fail until the PR I mentioned in the review is merged.

tool/github.sh Outdated

# Get packages for the test data.
(cd flutter-idea/testData/sample_tests; echo "pub get `pwd`"; pub get)
(cd flutter-idea/testData/sample_tests; echo "pub get `pwd`"; dart pub get)
Copy link
Member

Choose a reason for hiding this comment

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

This file has been fixed in a PR awaiting review. If you could add 'dart ' to the two echo commands it would make merging conflict-free after I get the review. Not mandatory, since I can resolve the conflict myself.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done!

tool/github.sh Outdated

# Set up the plugin tool.
(cd tool/plugin; echo "pub get `pwd`"; pub get)
(cd tool/plugin; echo "pub get `pwd`"; dart pub get)
Copy link
Member

Choose a reason for hiding this comment

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

ditto

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done!

@stevemessick stevemessick mentioned this pull request Mar 24, 2022
8 tasks
@stevemessick
Copy link
Member

@godofredoc Do you not have signing configured for this repo? If you don't want to bother, let me know. I can dup this PR and get you to review it, then merge.

@godofredoc
Copy link
Contributor Author

ning configured for this repo? If you don't want to bother, let me know. I can dup this PR and get you to review it, then merge.

I believe I do, let me send an update.

@godofredoc godofredoc merged commit 13c6836 into flutter:master Apr 5, 2022
@godofredoc godofredoc deleted the enable_scorecards branch April 5, 2022 01:50
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