Skip to content

Comments

Move permissions block after on block in test.yml for consistency#18

Merged
Sorkai merged 2 commits intoalert-autofix-10from
copilot/sub-pr-17
Dec 2, 2025
Merged

Move permissions block after on block in test.yml for consistency#18
Sorkai merged 2 commits intoalert-autofix-10from
copilot/sub-pr-17

Conversation

Copy link

Copilot AI commented Dec 2, 2025

Addresses review feedback on #17 to maintain consistent workflow file structure across the repository.

  • Moved permissions: block after on: block in .github/workflows/test.yml to match the pattern used in ci.yml and release.yml
# Before
name: Extension Tests
permissions:
  contents: read

on:
  push:
    branches: [ main ]

# After
name: Extension Tests

on:
  push:
    branches: [ main ]

permissions:
  contents: read

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Sorkai <35844770+Sorkai@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on permissions in code scanning alert fix Move permissions block after on block in test.yml for consistency Dec 2, 2025
Copilot AI requested a review from Sorkai December 2, 2025 16:40
@Sorkai Sorkai marked this pull request as ready for review December 2, 2025 16:42
@Sorkai Sorkai merged commit 6e9002a into alert-autofix-10 Dec 2, 2025
@Sorkai Sorkai deleted the copilot/sub-pr-17 branch December 2, 2025 16:42
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.

2 participants