Skip to content

Instruct rpminspect to ignore xml jinja templates#4635

Merged
LecrisUT merged 2 commits intomainfrom
psss-rpminspect-and-jinja
Mar 12, 2026
Merged

Instruct rpminspect to ignore xml jinja templates#4635
LecrisUT merged 2 commits intomainfrom
psss-rpminspect-and-jinja

Conversation

@psss
Copy link
Contributor

@psss psss commented Mar 3, 2026

The _base.xml.j2 template is incorrectly detected as xml and then always reported as a problem when releasing a new tmt in Fedora. Let's get rid of the false warning.

@psss psss added the packaging Changes related to the rpm packaging label Mar 3, 2026
@psss psss added this to planning Mar 3, 2026
@github-project-automation github-project-automation bot moved this to backlog in planning Mar 3, 2026
@psss psss moved this from backlog to review in planning Mar 3, 2026
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

The change adds an ignore rule to rpminspect.yaml for Jinja XML templates to prevent false warnings. One suggestion is provided to make the path glob more specific.

psss added 2 commits March 4, 2026 19:05
The `_base.xml.j2` template is incorrectly detected as `xml` and
then always reported as a problem when releasing a new `tmt` in
Fedora. Let's get rid of the false warning.
@psss psss force-pushed the psss-rpminspect-and-jinja branch from e6c9800 to 45fe574 Compare March 4, 2026 18:05
@happz happz moved this from review to merge in planning Mar 10, 2026
@psss
Copy link
Contributor Author

psss commented Mar 12, 2026

Let's get rid of those irrelevant failures, proposing for the current sprint.

@happz
Copy link
Contributor

happz commented Mar 12, 2026

Let's get rid of those irrelevant failures, proposing for the current sprint.

It shouldn't be even needed to be proposed for a sprint, it should be grabbed by the current release person when there's nothing better to get merged. It's reviewed, it's ready to get in. Successful story points, free of charge.

Edit: added to agenda, this should be a task for the release person, "from time to time check merge orphans".

@LecrisUT
Copy link
Contributor

No change in code, just packaging file, merging

@LecrisUT LecrisUT merged commit 474293c into main Mar 12, 2026
18 checks passed
@LecrisUT LecrisUT deleted the psss-rpminspect-and-jinja branch March 12, 2026 14:23
@github-project-automation github-project-automation bot moved this from merge to done in planning Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packaging Changes related to the rpm packaging

Projects

Status: done

Development

Successfully merging this pull request may close these issues.

4 participants