Skip to content

.github: order allowed path, fix name, add missing - #33664

Merged
peterbarker merged 1 commit into
ArduPilot:masterfrom
khancyr:wf_tidy_tests
Aug 10, 2026
Merged

.github: order allowed path, fix name, add missing#33664
peterbarker merged 1 commit into
ArduPilot:masterfrom
khancyr:wf_tidy_tests

Conversation

@khancyr

@khancyr khancyr commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary

extracted from #33575
Order Tools removal path for easier matching

Classification & Testing (check all that apply and add your own)

  • Checked by a human programmer
  • Non-functional change
  • No-binary change
  • Infrastructure change (e.g. unit tests, helper scripts)
  • Automated test(s) verify changes (e.g. unit test, autotest)
  • Tested manually, description below (e.g. SITL)
  • Tested on hardware
  • Logs attached
  • Logs available on request

Description

@github-actions github-actions Bot added the github_actions Pull requests that update GitHub Actions code label Jul 7, 2026

@peterbarker peterbarker 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.

Just consider that if this was a PR that was just, ".github: correct CHDK-Script spelling in exclude-from-run lists" I would glance through it in a minute flat and hit "merge".

Similarly for "add camera gimbal tools from exclude-from-run lists", "add terrain-tools to exclude-from-run list", "sort exclude from run lists".

Yes, many PRs, but when you fix this PR I have to go through all of these changes again which is just a waste of time.

Comment thread .github/workflows/test_chibios.yml Outdated
@khancyr
khancyr force-pushed the wf_tidy_tests branch 2 times, most recently from e17f50c to 78b8a3e Compare August 10, 2026 08:07
@khancyr khancyr mentioned this pull request Aug 10, 2026
9 tasks

@peterbarker peterbarker 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.

Near enough.

- 'Tools/environment_install/**'
- 'Tools/geotag/**'
- 'Tools/gittools/**'
- 'Tools/mavproxy_modules/**'

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.

This was never re-inserted.

- 'Tools/environment_install/**'
- 'Tools/geotag/**'
- 'Tools/gittools/**'
- 'Tools/mavproxy_modules/**'

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.

This was never reinserted

@peterbarker
peterbarker merged commit 09a5ce4 into ArduPilot:master Aug 10, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants