Skip to content

Conversation

@rocketeerbkw
Copy link
Member

@rocketeerbkw rocketeerbkw commented Feb 6, 2025

General Checklist

  • Affected Issues have been mentioned in the Closing issues section
  • Documentation has been written/updated
  • PR title is ready for inclusion in changelog
  • Added --organization-name to add/update variable
    • Changes are backwards compatible with older lagoon API
  • Added list organization-variables

Blocked on uselagoon/machinery#80.

@shreddedbacon
Copy link
Member

You can use go get github.com/uselagoon/machinery@basic-org-env-vars to add the machinery branch and push that so that this branch can be checked out and used locally. Just revert once the code is merged and released in machinery.

@shreddedbacon
Copy link
Member

v0.0.33 of machinery is released now, you can update this PR with that version if that is all that is needed to get this to be ready for review/release

@shreddedbacon shreddedbacon added this to the v0.32.0 milestone Mar 18, 2025
@rocketeerbkw rocketeerbkw marked this pull request as ready for review March 20, 2025 02:24
Copy link
Member

@shreddedbacon shreddedbacon left a comment

Choose a reason for hiding this comment

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

Works for me

@shreddedbacon shreddedbacon merged commit 0130db7 into main May 8, 2025
8 checks passed
@shreddedbacon shreddedbacon deleted the basic-org-env-vars branch May 8, 2025 22:36
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.

3 participants