Skip to content

Conversation

@Excoriate
Copy link
Owner

  • Fixed issues with CI workflows
  • Resolved test failures

Update the branch protection rules for the main and master branches to align with the Linux kernel's best practices:

- Require at least one approving review from a code owner before merging.
- Dismiss stale reviews when new commits are pushed.
- Require all required status checks to pass (e.g., Lint, Test).
- Enforce a linear commit history.
- Restrict push access to the maintainers team and specific users.
- Require signed commits.

The changes ensure that all contributions to the main and master branches go through a rigorous review process, maintain a clean commit history, and are signed by the author. This helps to uphold the quality and integrity of the codebase.

Affected files:
- .github/settings.yml

No functional changes to the application code. This is a purely administrative update to the repository configuration.
Update the GitHub Actions workflows to improve the labeling and
sizing functionality for pull requests.

The key changes are:

- Rename the "Assign labels on PR" workflow to a more descriptive
  name "🔖 Assign labels on PR".
- Update the trigger event to `pull_request_target` instead of
  `pull_request` to ensure the workflow can access the full
  context of the pull request, including labels.
- Upgrade the `actions/labeler` action to the latest version (v5).
- Add the `sync-labels` option to ensure the workflow keeps the
  repository labels in sync with the configuration in `.github/labeler.yml`.
- Update the `.github/labeler.yml` configuration to use the new
  `changed-files` syntax for more precise matching of file paths.
  This includes adding new patterns for scripts, config files, and
  meta-information like licenses and READMEs.

These changes improve the reliability and flexibility of the
automated labeling process for pull requests, ensuring that
changes are properly categorized according to the project's
conventions.

Affected files:
- .github/workflows/labels-assigner.yml
- .github/labeler.yml

No functional changes to the application code. This is purely an
update to the CI/CD tooling to enhance the developer experience
and streamline the pull request review process.
@Excoriate Excoriate merged commit 6ad5cbf into master Apr 28, 2025
2 checks passed
@Excoriate Excoriate deleted the ci/fix-workflows branch April 28, 2025 06:57
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.

2 participants