Skip to content

DO NOT MERGE#456

Closed
lsm5 wants to merge 3 commits intocontainers:mainfrom
lsm5:dnm-gha
Closed

DO NOT MERGE#456
lsm5 wants to merge 3 commits intocontainers:mainfrom
lsm5:dnm-gha

Conversation

@lsm5
Copy link
Copy Markdown
Member

@lsm5 lsm5 commented Apr 21, 2026

Summary by Sourcery

Documentation:

  • Normalize markdown formatting for the SELinux policy blog links in the README.

Signed-off-by: Lokesh Mandvekar <[email protected]>
@sourcery-ai
Copy link
Copy Markdown

sourcery-ai Bot commented Apr 21, 2026

Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

This PR performs minor formatting cleanup in the README’s SELinux policy blog links section and appends a temporary 'DO NOT MERGE' notice at the end of the document.

File-Level Changes

Change Details Files
Normalize markdown link formatting and spacing in the SELinux blog links list.
  • Removed trailing double-space line breaks from each blog link line to standardize markdown formatting
  • Kept link text and URLs identical while adjusting whitespace only
  • Ensured descriptions remain on the following lines without extra trailing spaces
README.md
Add an explicit 'DO NOT MERGE' notice to the README.
  • Inserted a new '## DO NOT MERGE' section heading at the end of the README
  • Left existing content unchanged aside from the new section insertion
README.md

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link
Copy Markdown
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

This pull request removes trailing whitespace from several lines in the README.md file and adds a 'DO NOT MERGE' header. A review comment points out that the 'DO NOT MERGE' header appears to be a development placeholder and should be removed to maintain clean documentation.

Comment thread README.md
**[Containers and MLS](https://danwalsh.livejournal.com/77830.html)**
Container-selinux policy support of MLS (Multi Level Security).

## DO NOT MERGE
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.

medium

The header ## DO NOT MERGE appears to be a development placeholder. It should be removed before this pull request is finalized to maintain clean documentation.

Change from v2 tag to main branch commit (69ef6d0) following the
pattern from sclorg/tfaga-wrapper.

Signed-off-by: Lokesh Mandvekar <[email protected]>
@packit-as-a-service
Copy link
Copy Markdown

Ephemeral COPR build failed. @containers/packit-build please check.

Signed-off-by: Lokesh Mandvekar <[email protected]>
@lsm5 lsm5 closed this Apr 21, 2026
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.

1 participant