Skip to content

.github: labels-by-language remove extraneous string "git,"#32750

Merged
tpwrules merged 1 commit into
ArduPilot:masterfrom
cclauss:labels-by-language-remove-sting-git
Apr 12, 2026
Merged

.github: labels-by-language remove extraneous string "git,"#32750
tpwrules merged 1 commit into
ArduPilot:masterfrom
cclauss:labels-by-language-remove-sting-git

Conversation

@cclauss

@cclauss cclauss commented Apr 12, 2026

Copy link
Copy Markdown
Contributor

Summary

Error: Unhandled error: ReferenceError: git is not defined

The listFiles API call has an extra git, after context.repo.repo, which JavaScript interprets as a reference to an undefined variable named git.

Note

One of the strange things about failing GitHub Actions pull_request_trigger jobs is that they will continue to fail until after they are merged. This is because pull_request_trigger jobs run on the master branch, not on the pull request branch.

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

@peterbarker

Copy link
Copy Markdown
Contributor

Please test this on your own repo and get back to us when it is known working and tested.

@cclauss

cclauss commented Apr 12, 2026

Copy link
Copy Markdown
Contributor Author

known working and tested.

@tpwrules tpwrules merged commit c49a68d into ArduPilot:master Apr 12, 2026
108 of 109 checks passed
@cclauss cclauss deleted the labels-by-language-remove-sting-git branch April 12, 2026 16:52
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