Skip to content

Fix Prettier formatting in spelling-all.yaml #39773

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
danieljurek merged 4 commits into from
Jan 20, 2026
Merged

Fix Prettier formatting in spelling-all.yaml #39773

danieljurek merged 4 commits into from
Jan 20, 2026

Conversation

Copy link
Contributor

Copilot AI commented Jan 20, 2026 •
edited
Loading

Prettier check failed due to quote style inconsistency in the newly added pull_request trigger paths.

Changes

  • Changed single quotes to double quotes in spelling-all.yaml path filters to match repository code style
pull_request:
  paths:
    - "eng/common/spelling/**"  # was: 'eng/common/spelling/**'
    - "cspell.json"              # was: 'cspell.json'

The repository's Prettier configuration enforces double quotes for YAML string values.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Add trigger for spelling changes in GitHub Actions Add path trigger for spelling tooling changes to SpellCheck - All workflow Jan 20, 2026
Copilot AI requested a review from danieljurek January 20, 2026 20:00
@github-actions
Copy link

github-actions bot commented Jan 20, 2026 •
edited
Loading

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ The required check named Protected Files has failed. Refer to the check in the PR's 'Checks' tab for details on how to fix it and consult the aka.ms/ci-fix guide


Comment generated by summarize-checks workflow run.

@danieljurek danieljurek moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 📆🎇 Jan 20, 2026
Copilot AI changed the title Add path trigger for spelling tooling changes to SpellCheck - All workflow Add PR trigger for spelling tooling changes to SpellCheck - All workflow Jan 20, 2026
@danieljurek danieljurek marked this pull request as ready for review January 20, 2026 20:27
@danieljurek danieljurek marked this pull request as draft January 20, 2026 20:28
Copilot AI changed the title Add PR trigger for spelling tooling changes to SpellCheck - All workflow Fix Prettier formatting in spelling-all.yaml Jan 20, 2026
Copilot AI requested a review from danieljurek January 20, 2026 20:34
@danieljurek danieljurek marked this pull request as ready for review January 20, 2026 21:52
@danieljurek danieljurek merged commit e4c6d21 into main Jan 20, 2026
73 of 92 checks passed
@danieljurek danieljurek deleted the copilot/add-trigger-for-spelling-check branch January 20, 2026 21:53
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 📆🎇 Jan 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@weshaggard weshaggard Awaiting requested review from weshaggard weshaggard is a code owner

@mikeharder mikeharder Awaiting requested review from mikeharder mikeharder is a code owner

@benbp benbp Awaiting requested review from benbp benbp is a code owner

@danieljurek danieljurek Awaiting requested review from danieljurek

Assignees

@danieljurek danieljurek

Copilot code review Copilot

Labels

None yet

Projects

Azure SDK EngSys 📆🎇
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@danieljurek