Skip to content

Merge beta to master #854

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

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

Merge beta to master #854

wants to merge 8 commits into from

Conversation

fgreinacher
Copy link
Contributor

@fgreinacher fgreinacher commented May 19, 2025

This brings the CI catalog changes from the beta channel to the release channel.

I have successfully verified the new feature: https://gitlab.com/gitlab-org/gitlab/-/issues/463253#note_2509219147

Contributes to #822

@fgreinacher fgreinacher force-pushed the beta branch 2 times, most recently from 6e5d4c1 to c128fc6 Compare May 19, 2025 20:26
@fgreinacher fgreinacher requested a review from Copilot May 19, 2025 20:32
Copy link

🎉 This PR is included in version 13.3.0-beta.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR merges CI catalog changes from the beta channel to the release channel. Key changes include adding a new "publishToCatalog" option to the configuration, updating tests to cover the catalog publishing logic, and incorporating necessary documentation and error definitions.

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
test/verify.test.js Added tests to verify error handling when GitLab CLI is missing
test/resolve-config.test.js Updated default configuration to include publishToCatalog
test/publish.test.js Added tests for successful and error scenarios when publishing to the CI catalog
package.json Updated dependencies to support the new functionality
lib/verify.js Added logic to verify GitLab CLI installation for catalog publishing
lib/resolve-config.js Added publishToCatalog to the resolved configuration
lib/publish.js Integrated glab CLI invocation for catalog publishing
lib/glab.js Created a wrapper for calling the glab CLI using execa
lib/definitions/errors.js Added a new error definition for missing GitLab CLI
README.md Documented the experimental publishToCatalog feature
Comments suppressed due to low confidence (1)

lib/verify.js:95

  • The glab function is used in this block but is not imported, which may lead to a runtime error when publishToCatalog is true. Please import glab from './glab.js' at the beginning of the file.
if (publishToCatalog === true) {

Copy link

🎉 This PR is included in version 13.3.0-beta.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@fgreinacher fgreinacher requested a review from JonasSchubert May 19, 2025 21:10
@fgreinacher
Copy link
Contributor Author

I had a chat with Furkan from GitLab today who brought up a few good ideas. Let's hold this PR back until the way forward is clearer.

@fgreinacher fgreinacher removed the request for review from JonasSchubert May 30, 2025 14:36
@fgreinacher fgreinacher marked this pull request as draft June 15, 2025 11:36
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.

1 participant