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.

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

Fix CI workflows #60

Merged
merged 5 commits into from
Dec 3, 2024
Merged

Fix CI workflows #60

merged 5 commits into from
Dec 3, 2024

Conversation

matrei
Copy link
Contributor

@matrei matrei commented Dec 3, 2024

This PR will:

  • Update the Git user for CI operations to the one set in the organization secrets. At present that user is gpc-deploy.
  • Switch to the correct secret for DEVELOCITY_ACCESS_KEY
  • Fix release-drafter.yml

…ration

The publishing process was broken, possibly due to the use of `PublicationContainer.register()`, which lazily configures publications.
This commit tries to resolve the issue by switching to `PublicationContainer.create()`, ensuring publications are eagerly configured during the build process.
Replace a personal GitHub account with the dedicated CI account for Git operations in GitHub Actions. This ensures commits and tags created during CI runs are properly attributed to the CI system.
@gpc-deploy gpc-deploy added the bug label Dec 3, 2024
@matrei matrei requested review from sbglasius and removed request for sbglasius December 3, 2024 10:21
@matrei matrei marked this pull request as draft December 3, 2024 10:23
@matrei matrei changed the title Fix publishing Update Git user to dedicated CI account Dec 3, 2024
The secret in this repo is named `DEVELOCITY_ACCESS_KEY`.
Error: Data doesn't contain `nodes` field.
@matrei matrei changed the title Update Git user to dedicated CI account Fix CI workflows Dec 3, 2024
@matrei matrei marked this pull request as ready for review December 3, 2024 10:51
@matrei matrei requested a review from sbglasius December 3, 2024 10:51
@matrei matrei merged commit c2ac465 into 4.0.x Dec 3, 2024
5 checks passed
@matrei matrei deleted the matrei/fix-publishing branch December 3, 2024 11:05
matrei added a commit that referenced this pull request Dec 3, 2024
* ci: Update Git user to dedicated CI account

Replace a personal GitHub account with the dedicated CI account for Git operations in GitHub Actions. This ensures commits and tags created during CI runs are properly attributed to the CI system.

* ci: Switch secret for `DEVELOCITY_ACCESS_KEY`

The secret in this repo is named `DEVELOCITY_ACCESS_KEY`.

* ci: Fix release-notes.yml

Error: Data doesn't contain `nodes` field.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants