Skip to content

feat(action): add action to create GCP Projects#13

Merged
tdabasinskas merged 1 commit intodatolabs-io:mainfrom
tfhartmann:adding-create-project
Mar 21, 2025
Merged

feat(action): add action to create GCP Projects#13
tdabasinskas merged 1 commit intodatolabs-io:mainfrom
tfhartmann:adding-create-project

Conversation

@tfhartmann
Copy link
Contributor

Allows for software templates to be created to create projects if the SA has permission to do so.

@tfhartmann tfhartmann requested a review from a team as a code owner March 21, 2025 13:26
@tfhartmann tfhartmann force-pushed the adding-create-project branch 2 times, most recently from abf074f to ca35bbb Compare March 21, 2025 13:36
@tdabasinskas tdabasinskas self-assigned this Mar 21, 2025
@tdabasinskas tdabasinskas added the enhancement New feature or request label Mar 21, 2025
Copy link
Member

@tdabasinskas tdabasinskas left a comment

Choose a reason for hiding this comment

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

Hi @tfhartmann,

Appreciate the contribution! I've fixed the issues with failing pipeline and also added few minor comments. If you could address these, I'm more than happy to get this merged.

@tdabasinskas tdabasinskas enabled auto-merge (squash) March 21, 2025 16:51
auto-merge was automatically disabled March 21, 2025 17:36

Head branch was pushed to by a user without write access

@tfhartmann tfhartmann force-pushed the adding-create-project branch 2 times, most recently from 4dda450 to 61b7f95 Compare March 21, 2025 17:48
@tfhartmann tfhartmann requested a review from tdabasinskas March 21, 2025 17:48
@tfhartmann
Copy link
Contributor Author

Thanks @tdabasinskas hows this look?

@tfhartmann tfhartmann force-pushed the adding-create-project branch from 61b7f95 to b513f19 Compare March 21, 2025 17:58
Allows for software templates to be created to create projects if the
SA has permission to do so.

docs(readme): update GCP project creation instructions and add usage example

fix(gcp): remove unnecessary console log in create project action

refactor: implement pr feedback

fix(deps): dedup yarn.lock

fix(prettier): fixing formatting
@tfhartmann tfhartmann force-pushed the adding-create-project branch from b513f19 to fa55854 Compare March 21, 2025 17:59
@tdabasinskas tdabasinskas enabled auto-merge (squash) March 21, 2025 18:10
Copy link
Member

@tdabasinskas tdabasinskas left a comment

Choose a reason for hiding this comment

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

Looks great - thanks again for a contribution! 🚀

@tdabasinskas tdabasinskas disabled auto-merge March 21, 2025 18:11
@tdabasinskas tdabasinskas merged commit b381c4f into datolabs-io:main Mar 21, 2025
8 checks passed
@tfhartmann tfhartmann deleted the adding-create-project branch March 21, 2025 18:14
@tfhartmann
Copy link
Contributor Author

Looks great - thanks again for a contribution! 🚀

no problem! i've actually a few more in mind too 😄

@tfhartmann
Copy link
Contributor Author

@tdabasinskas it looks like the release didn't work? 😢

@tdabasinskas
Copy link
Member

@tdabasinskas it looks like the release didn't work? 😢

Thanks for a heads-up, fixed - https://www.npmjs.com/package/@datolabs/plugin-scaffolder-backend-module-gcp/v/0.2.0.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants