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:
This PR changes the artifact paths from ghcr.io/openmcp-project/github.com/openmcp-project/... to ghcr.io/openmcp-project/....

Release note:

Copy link
Contributor

@Diaphteiros Diaphteiros left a comment

Choose a reason for hiding this comment

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

See remarks about documentation index generation, looks good otherwise.

Taskfile.yaml Outdated
CODE_DIRS: '{{.ROOT_DIR}}/cmd/... {{.ROOT_DIR}}/internal/... {{.ROOT_DIR}}/test/... {{.ROOT_DIR}}/api/core/v1alpha1/...'
COMPONENTS: 'project-workspace-operator'
REPO_URL: 'https://github.com/openmcp-project/project-workspace-operator'
GENERATE_DOCS_INDEX: "true"
Copy link
Contributor

Choose a reason for hiding this comment

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

We should either remove this, or set it up correctly, otherwise it just generates an empty index file.

See https://github.com/openmcp-project/build?tab=readme-ov-file#documentation-index-generation for a short documentation on the feature and https://github.com/openmcp-project/mcp-operator/blob/main/docs/controllers/.docnames for an example of one of the mentioned metadata files.

docs/README.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

We should either remove the documentation index generation and this file or set it up to contain actual content, see above.

moelsayed and others added 2 commits May 1, 2025 23:16
Copy link
Contributor

@Diaphteiros Diaphteiros 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 c17e034 into openmcp-project:main May 6, 2025
3 checks passed
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.

2 participants