Skip to content

Fix sibling GitHub Actions job scopes#52

Merged
keith-hall merged 1 commit into
SublimeText:masterfrom
floze-the-genius:fix/51-job-scopes
Jul 18, 2026
Merged

Fix sibling GitHub Actions job scopes#52
keith-hall merged 1 commit into
SublimeText:masterfrom
floze-the-genius:fix/51-job-scopes

Conversation

@floze-the-genius

Copy link
Copy Markdown
Contributor

Summary

  • leave the current job context when indentation returns to the jobs mapping level
  • let expect-jobs classify every sibling job identifier consistently
  • add a regression assertion for the existing second job fixture

Fixes #51.

Tests

  • Sublime syntax test build 4205 with default Packages master and PowerShell master (focused GitHub Actions package): pass
  • Sublime syntax test build 4200 with default Packages v4200 and PowerShell st4107-5.1.1 (focused GitHub Actions package): pass
  • git diff --check: pass

The all-package local Docker invocation also reached and passed the changed GitHub Actions test; unrelated existing Drone/Kong assertions were not used as the change gate because the ad hoc runner did not resolve Kong's source.yaml.sublime.syntax dependency. The repository's scheduled upstream workflow is green, and PR CI remains the authoritative full-suite result.

Disclosure

OpenAI Codex was used to investigate, implement, and test this change. The commit includes a DCO sign-off.

Pop the job-definition context when indentation returns to the job level so every job identifier is parsed by expect-jobs. Add a syntax assertion for the existing second job.

Signed-off-by: Floze <88098863+floze-the-genius@users.noreply.github.com>

@keith-hall keith-hall left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks

@keith-hall
keith-hall merged commit cfa12ff into SublimeText:master Jul 18, 2026
2 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.

[GHA] Inconsistent entity.name.function.github-actions

2 participants