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.

Sign up for GitHub

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

Jump to bottom

fix #129 ensure gcpmanager is included by default #131

Merged
merged 1 commit into from
Jul 23, 2023
Merged

fix #129 ensure gcpmanager is included by default #131

merged 1 commit into from
Jul 23, 2023

Conversation

samos123
Copy link
Contributor

No description provided.

@samos123 samos123 requested review from nstogner and brandonjbjelland and removed request for nstogner July 23, 2023 06:14
brandonjbjelland
brandonjbjelland approved these changes Jul 23, 2023
View reviewed changes
Copy link
Contributor

@brandonjbjelland brandonjbjelland left a comment

Choose a reason for hiding this comment

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

💯

@samos123 samos123 merged commit 4a58efe into main Jul 23, 2023
2 checks passed
@samos123 samos123 deleted the fix-129-missing-gcp-manager-config branch July 23, 2023 06:52
@samos123
Copy link
Contributor Author

Thanks for pairing on this! This was coauthored by @brandonjbjelland and forgot to add you as a coauthor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brandonjbjelland brandonjbjelland brandonjbjelland approved these changes

@nstogner nstogner Awaiting requested review from nstogner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@samos123 @brandonjbjelland