-
Notifications
You must be signed in to change notification settings - Fork 336
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
DevTools License and 3P Notice Review #8216
Comments
mossmana
added
d: devtools
P1
high priority issues at the top of the work list, actively being worked on.
labels
Aug 20, 2024
mossmana
added a commit
that referenced
this issue
Nov 7, 2024
Created a utility for updating licenses in configured source files that can be run via tests and/or standalone. #8216 * Initial YAML config parsing logic and tests * added additional YAML config parsing logic * Added comments and some logic to parse stored value * renamed functions to better match functionality, added logic to return replacement header, and added more tests * added support for bulk update * added test for full repo check for incorrect licenses * ran dart formatter * fixed missing training commas * moved license utils to tool * fixed dependencies * added github action to run tool tests * added write permission to tool_tests.sh * fixed Dartdoc function comment, experimenting with repo wide check test on CI * excluding flutter-sdk dependencies from repo wide check * test that the repo wide check actually catches incorrect license header * Turned off repo wide check test * addressed PR feedback * addressed more PR feedback * reverted some accidentally changed files, addressed additional PR feedback * fixed DartDoc and other PR feedback * added dry run option for bulk update * corrected logic for test * fixed missing trailing comma * addressing some missed PR feedback * removed late and added back final to member variables * added clearer docs and removed unnecessary todo * addressed missed PR feedback * addressed another round of PR feedback * fixed git issues * additional PR feedback * another round of PR feedback
7 tasks
7 tasks
This was referenced Jan 7, 2025
This was referenced Jan 15, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
e.g. Here's what the help menu option looks like in VS Code (just a link that redirects to https://code.visualstudio.com/license?lang=en)
The text was updated successfully, but these errors were encountered: