Skip to content

Comments

Migrate generate_code.sh to devtools_tool for Windows compatibility#6579

Merged
auto-submit[bot] merged 6 commits intoflutter:masterfrom
DanTup:migrate-generate-code
Oct 24, 2023
Merged

Migrate generate_code.sh to devtools_tool for Windows compatibility#6579
auto-submit[bot] merged 6 commits intoflutter:masterfrom
DanTup:migrate-generate-code

Conversation

@DanTup
Copy link
Contributor

@DanTup DanTup commented Oct 24, 2023

On Windows I've had to run build_runner build manually in each folder so this adds a devtools_tool command in Dart and calls it from the old bash script.

@@ -10,37 +10,9 @@ RELATIVE_PATH_TO_SCRIPT="${BASH_SOURCE[0]}"
# The directory that this script is located in.
TOOL_DIR=`dirname "${RELATIVE_PATH_TO_SCRIPT}"`
Copy link
Member

Choose a reason for hiding this comment

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

lets just delete this file altogether and call devtools_tool generate-code everywhere we were previously calling generate_code.sh

Screenshot 2023-10-24 at 9 25 51 AM

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! I wasn't sure if others might have scripts/keybinds to call it, but this probably makes more sense. I'll ping a note in Discord about it too.

@DanTup DanTup requested a review from a team as a code owner October 24, 2023 16:42
Copy link
Member

@kenzieschmoll kenzieschmoll left a comment

Choose a reason for hiding this comment

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

one comment then lgtm

@DanTup DanTup added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 24, 2023
@auto-submit auto-submit bot merged commit b6d384f into flutter:master Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App release-notes-not-required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants