Skip to content

feat(scripts): Add script — [Lua App Submission] EdgeTX Flight Log - #68

Merged
pfeerick merged 3 commits into
mainfrom
add-script/61
Aug 4, 2026
Merged

feat(scripts): Add script — [Lua App Submission] EdgeTX Flight Log#68
pfeerick merged 3 commits into
mainfrom
add-script/61

Conversation

@github-actions

@github-actions github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Automated PR for script submission from issue #61.

Checklist before merging

  • Verify the scripts.json diff looks correct
  • If screenshots are available locally, add them to ASSETS/ and update image paths in scripts.json
  • Remove draft status when ready to merge

Closes #61

pfeerick added a commit that referenced this pull request Aug 4, 2026
…#69)

The push job in validate-issue-templates.yml was supposed to self-heal
bot-created branches, but script-submission.yml's branch-creating push uses
the default GITHUB_TOKEN, and GitHub Actions suppresses new workflow-
triggering events (including push) for GITHUB_TOKEN-authored pushes — the
same rule that prevents the sync job's own corrective commits from causing
an infinite loop also silently prevented it from ever running in the first
place for these branches. Confirmed against two real submissions (#67, #68):
both landed with a permanently-failing check and no auto-fix.

Rather than fight the retrigger suppression, run the sync script directly
in script-submission.yml before committing, so the schema/templates are
already correct in the very first commit of every bot-generated PR. The
push job in validate-issue-templates.yml remains as a backstop for
genuine human-authored pushes (e.g. direct-edit PRs), which aren't subject
to this suppression.

Also fixes a stale error-comment string referencing the category enum
check that was removed in #65.

Co-authored-by: Peter Feerick <5500713+pfeerick@users.noreply.github.com>
Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
[Lua App Submission] EdgeTX Flight Log

Closes #61
@pfeerick
pfeerick marked this pull request as ready for review August 4, 2026 01:54
@pfeerick
pfeerick merged commit 4ec489e into main Aug 4, 2026
12 checks passed
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.

[Lua App Submission] EdgeTX Flight Log

1 participant