Skip to content
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

tweaks to convert-go.sh #11762

Conversation

c2thorn
Copy link
Member

@c2thorn c2thorn commented Sep 19, 2024

convert-go.sh runs in root directory, not in mmv1/ folder. Makes it consistent with cherry-pick.sh
Also can handle relative or absolute template files.

Directions are now:

scripts/convert-go.sh does the conversion in a pre-switchover commit and outputs the converted files to *.temp files.
Example command in the root MM directory (in a pre-switchover commit)

sh scripts/convert-go.sh <provider output directory> <comma separated list of files that have changed in the PR> 

scripts/cherry-pick.sh prepares a new branch in a post-switchover commit and cherry-picks the previous *.temp files
Example command from root MM directory

sh scripts/cherry-pick <hash of a post-switchover commit>

@c2thorn c2thorn force-pushed the rewrite-convert-scripts branch from 5823caa to 8942909 Compare September 19, 2024 21:12
@c2thorn c2thorn merged commit 935e62d into GoogleCloudPlatform:rewrite-script-updates Sep 19, 2024
3 of 4 checks passed
@c2thorn c2thorn deleted the rewrite-convert-scripts branch September 19, 2024 21:22
@c2thorn c2thorn restored the rewrite-convert-scripts branch September 19, 2024 21:42
c2thorn added a commit to c2thorn/magic-modules that referenced this pull request Sep 24, 2024
…tchover-preview

* rewrite-script-updates:
  remove newline
  remove "already modified" error
  rewrite - script tweaks to replace path and QoL branching
  tweaks to convert-go.sh (GoogleCloudPlatform#11762)
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.

1 participant