Skip to content

Conversation

@moelsayed
Copy link
Contributor

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

@moelsayed moelsayed force-pushed the ocm_action branch 11 times, most recently from a286dcb to 2f51027 Compare March 7, 2025 16:58
@@ -1,6 +1,6 @@
apiVersion: v2
name: btpflm-project-workspace-operator
Copy link
Member

Choose a reason for hiding this comment

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

not sure about the btpflm thing here

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, let's change that to e.g. openmcp-project-workspace-operator.

Copy link
Member

Choose a reason for hiding this comment

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

Or just project-workspace-operator if the path of the Helm chart (e.g. for its installation) is sufficient enough.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@maximiliantech @maximilianbraun updated, please check.

Copy link
Member

@maximiliantech maximiliantech left a comment

Choose a reason for hiding this comment

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

Could you please have a look on the comment below, otherwise LGTM!

Copy link
Member

@maximiliantech maximiliantech left a comment

Choose a reason for hiding this comment

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

LGTM

@moelsayed moelsayed merged commit d4246a4 into main Mar 10, 2025
6 checks passed
@maximiliantech maximiliantech deleted the ocm_action branch March 12, 2025 12:29
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.

4 participants