Skip to content

Fix properties from GitLab "descrpition" and "protectedDefaultBranch"#111

Merged
JslYoon merged 3 commits into
redhat-ai-dev:mainfrom
JslYoon:JslYoon-fix-gitlab
May 26, 2026
Merged

Fix properties from GitLab "descrpition" and "protectedDefaultBranch"#111
JslYoon merged 3 commits into
redhat-ai-dev:mainfrom
JslYoon:JslYoon-fix-gitlab

Conversation

@JslYoon

@JslYoon JslYoon commented May 20, 2026

Copy link
Copy Markdown
Contributor

…tError

The publish:gitlab action does not support 'description' and 'protectDefaultBranch' properties, causing all templates to fail when publishing to GitLab. These properties are supported by publish:github but not by the GitLab scaffolder backend.

What does this PR do?:

Changes the yaml structure of the fields "description and "protectedDefaultBranch" on gitlab that was causing an error when building software template with gitlab.

Which issue(s) this PR fixes:

https://redhat.atlassian.net/browse/RHDHBUGS-3162

PR acceptance criteria:

Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened and linked to this PR, if they are not in the PR scope due to various constraints.

  • Tested and Verified

  • Documentation (READMEs, Product Docs, Blogs, Education Modules, etc.)

How to test changes / Special notes to the reviewer:

…tError

The publish:gitlab action does not support 'description' and 'protectDefaultBranch'
properties, causing all templates to fail when publishing to GitLab. These properties
are supported by publish:github but not by the GitLab scaffolder backend.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@JslYoon JslYoon requested a review from a team as a code owner May 20, 2026 20:10
Update skeleton/template.yaml to remove 'description' and 'protectDefaultBranch'
from publish:gitlab actions. This ensures generate.sh produces templates that
match the expected output and don't cause CI failures.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
yangcao77
yangcao77 previously approved these changes May 25, 2026

@Jdubrick Jdubrick left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Signed-off-by: Lucas <lyoon@redhat.com>
@JslYoon

JslYoon commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

Should we change it to settings.description?

https://backstage.io/api/stable/functions/_backstage_plugin-scaffolder-backend-module-gitlab.createPublishGitlabAction.html?

You're right, I've modified and verified it's working. thank you for pointing it out

@JslYoon JslYoon requested a review from yangcao77 May 26, 2026 18:42
@JslYoon JslYoon changed the title Remove unsupported properties from GitLab publish actions to fix Inpu… Fix unsupported properties from GitLab publish actions to fix Inpu… May 26, 2026

@Jdubrick Jdubrick left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@JslYoon JslYoon changed the title Fix unsupported properties from GitLab publish actions to fix Inpu… Fix properties from GitLab "descrpition" and "protectedDefaultBranch" May 26, 2026
@JslYoon JslYoon merged commit d932732 into redhat-ai-dev:main May 26, 2026
2 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.

3 participants