Skip to content

Conversation

@jrosco
Copy link
Contributor

@jrosco jrosco commented Dec 11, 2025

Fixes an issue with Git commands not knowing which user to push version bumps for on-release events. This will now bump the version with correct user and commit back to the working branches.

Adds the GitHub inject environment vars so these can be passed to the housekeeping script.

@jrosco jrosco force-pushed the fix/housekeeping_script_on_release branch from e22738b to 8cdbb92 Compare December 11, 2025 02:58
Fixes an issue with Git commands not knowing which user to push version
bumps for on-release events. This will now bump the version with correct
user and commit back to the working branches.

Adds the GitHub inject environment vars so these can be passed to the
housekeeping script.
@jrosco jrosco force-pushed the fix/housekeeping_script_on_release branch from 3c325c6 to a6d4d63 Compare December 11, 2025 03:00
@jrosco jrosco marked this pull request as ready for review December 11, 2025 03:00
@jrosco jrosco requested a review from a team as a code owner December 11, 2025 03:00
@qctrlrobot qctrlrobot requested a review from a team as a code owner December 11, 2025 03:01
Comment on lines +22 to +24
- name: Inject environment variables
run: |
./ci env prepareGitHub
Copy link
Member

Choose a reason for hiding this comment

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

question: Is this just setting GITHUB_TOKEN as an environment variable from here so it can then be used by this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@LimbicNode42 Adds enviroments variable that can used in our scripts. See a example job run here

image

@jrosco jrosco requested a review from LimbicNode42 December 11, 2025 03:31
@jrosco jrosco merged commit 24a70cb into master Dec 11, 2025
13 checks passed
@jrosco jrosco deleted the fix/housekeeping_script_on_release branch December 11, 2025 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants