Skip to content

github/workflows: remove pull_request branches selection#1998

Merged
dlech merged 1 commit into
hbldh:developfrom
JPHutchins:github/run-ci-on-all-prs
Jun 12, 2026
Merged

github/workflows: remove pull_request branches selection#1998
dlech merged 1 commit into
hbldh:developfrom
JPHutchins:github/run-ci-on-all-prs

Conversation

@JPHutchins

Copy link
Copy Markdown
Contributor

GitHub imposes no limit on on actions usage by public repositories, so filtering of PRs is removed to aid developers who branch and PR to/from arbitrary branches.

GitHub imposes no limit on on actions usage by public
repositories, so filtering of PRs is removed to aid developers
who branch and PR to/from arbitrary branches.
Copilot AI review requested due to automatic review settings June 12, 2026 19:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates GitHub Actions workflow triggers to broaden when CI runs on pull requests.

Changes:

  • Removed pull_request.branches filters from the format/lint workflow.
  • Removed pull_request.branches filters from the build/test workflow (keeping paths filtering).

Reviewed changes

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

File Description
.github/workflows/format_and_lint.yml Removes branch scoping for PR-triggered formatting/linting runs.
.github/workflows/build_and_test.yml Removes branch scoping for PR-triggered build/test runs while still restricting by paths.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/format_and_lint.yml
Comment thread .github/workflows/build_and_test.yml
@codecov

codecov Bot commented Jun 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.47%. Comparing base (ae2f589) to head (86fcc16).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1998      +/-   ##
===========================================
+ Coverage    52.45%   52.47%   +0.02%     
===========================================
  Files           43       43              
  Lines         4097     4097              
  Branches       504      504              
===========================================
+ Hits          2149     2150       +1     
  Misses        1817     1817              
+ Partials       131      130       -1     
Flag Coverage Δ
bluez-integration-py310 39.15% <ø> (ø)
bluez-integration-py311 39.15% <ø> (ø)
bluez-integration-py312 39.15% <ø> (ø)
bluez-integration-py313 39.15% <ø> (ø)
bluez-integration-py314 37.67% <ø> (+0.02%) ⬆️
macos-latest-py310 20.03% <ø> (ø)
macos-latest-py311 20.03% <ø> (ø)
macos-latest-py312 20.03% <ø> (ø)
macos-latest-py313 20.03% <ø> (ø)
macos-latest-py314 19.84% <ø> (ø)
ubuntu-latest-py310 23.94% <ø> (ø)
ubuntu-latest-py311 23.94% <ø> (ø)
ubuntu-latest-py312 23.94% <ø> (ø)
ubuntu-latest-py313 23.94% <ø> (ø)
ubuntu-latest-py314 22.05% <ø> (ø)
windows-latest-py310 18.72% <ø> (ø)
windows-latest-py311 18.72% <ø> (ø)
windows-latest-py312 18.72% <ø> (ø)
windows-latest-py313 18.72% <ø> (ø)
windows-latest-py314 18.44% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@dlech dlech merged commit b441623 into hbldh:develop Jun 12, 2026
23 checks passed
@dlech

dlech commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Seems reasonable. Thanks.

@JPHutchins JPHutchins deleted the github/run-ci-on-all-prs branch June 12, 2026 20:37
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