-
Notifications
You must be signed in to change notification settings - Fork 2
Go update and workflow fixes (#1) #1
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
Conversation
Fork of workspace manager that was originally created by Tom while working at MoJ. This updates go to 1.21 and goreleaser as well as replacing the workflows with a combined semver tag and release workflow. * update go packages * update go releaser config * update release workflow * localised readme changes
| release: | ||
| prerelease: auto | ||
| github: | ||
| owner: TomTucka |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed repo details
| @@ -1,12 +1,13 @@ | |||
| project_name: terraform-workspace-manager | |||
| project_name: opg-terraform-workspace-manager | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed project name
|
|
||
| env_files: | ||
| github_token: ${{ secrets.access_token }} | ||
| github_token: ${{ secrets.GITHUB_TOKEN }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed secret
| @@ -1,5 +1,6 @@ | |||
| # terraform-workspace-cleanup | |||
|
|
|||
| Forked from [this code base](https://github.com/TomTucka/terraform-workspace-manager). | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed path and added note about fork
|
hey @TomTucka - i made this a draft as it contains changes (i've put comments for most) that relate to our specific fork that may not want to include It also changes the workflow to automatically tag and create a release for both pr & push to main (we've renamed master to main in our fork) |
* Add renovate.json * Add configuration for OPG Digital conventions and policy --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: ramchale <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
prPriority is not valid in `major` and `vulnerabilityAlerts` The documentation states "Renovate always creates security PRs, even if the concurrent PR limit is already reached.", so removing these does not introduce any additional risks.
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Fork of workspace manager that was originally created by Tom while working at MoJ.
This updates go to 1.21 and goreleaser as well as replacing the workflows with a combined semver tag and release workflow.