Skip to content

update.sh script invoked by main branch workflow needs to be in vNext#558

Merged
wraschke merged 4 commits into
WASdev:vNextfrom
wraschke:update-script-needed-vNext
Oct 31, 2023
Merged

update.sh script invoked by main branch workflow needs to be in vNext#558
wraschke merged 4 commits into
WASdev:vNextfrom
wraschke:update-script-needed-vNext

Conversation

@wraschke

Copy link
Copy Markdown
Member

The update.sh script is invoked by the workflow file being introduced under #557. That workflow checks out the vNext branch, so the script must be present there. Otherwise, a failure like this will occur in workflow:

bash: ./update.sh: No such file or directory
Error: Process completed with exit code 127.

@wraschke wraschke force-pushed the update-script-needed-vNext branch from 623c601 to 5e15041 Compare October 30, 2023 19:20

@leochr leochr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@wraschke Looks good. Just one comment about the file name and directory. Thanks

Comment thread update.sh Outdated
@@ -0,0 +1,72 @@
#!/bin/bash

echo "Hello from the update.sh script!"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'll suggest adding this script under a folder: internal or ci
Suggestion for file name: "create-new-release.sh"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@leochr, I renamed the file and placed in the "internal" directory. Please take another look.

@wraschke wraschke force-pushed the update-script-needed-vNext branch from 697a114 to 69f8047 Compare October 31, 2023 15:56
leochr
leochr previously approved these changes Oct 31, 2023
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