Skip to content

Conversation

@suyask-msft
Copy link
Collaborator

This pull request makes a minor adjustment to the CI pipeline configuration by enforcing stricter formatting checks. The change removes the continueOnError: true flag from the formatting step, so formatting errors will now cause the build to fail instead of being ignored.

  • CI pipeline enforcement:
    • .azdo/ci-pr.yaml: Removed continueOnError: true from the Black formatting check, making formatting errors block the build.

suyask-msft and others added 8 commits November 17, 2025 09:23
- Changed complete_walkthrough.py to walkthrough.py in all references
- Swapped API reference and Product documentation URLs in header
- Cleaned up limitations section for clarity
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- Remove continue-on-error flag from Black formatting step
- Black formatting is now a blocking gate for PRs
- All code has been formatted to pass Black checks
- Remove continueOnError flag from Black check in ci-pr.yaml
- Makes Black formatting a blocking gate in Azure DevOps PRs
- Matches GitHub Actions workflow configuration
Copilot AI review requested due to automatic review settings November 17, 2025 19:34
@suyask-msft suyask-msft requested a review from a team as a code owner November 17, 2025 19:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enforces stricter code quality standards by making Black formatting checks mandatory in the Azure DevOps CI pipeline, aligning it with the existing GitHub Actions workflow behavior.

  • Removed continueOnError: true from the Black formatting step, making formatting violations fail the build
  • This change brings the Azure DevOps pipeline into alignment with the GitHub Actions workflow, which already enforces Black formatting

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zhaodongwang-msft zhaodongwang-msft merged commit c14d2c9 into main Nov 17, 2025
14 checks passed
@suyask-msft suyask-msft deleted the feature/enforce-black-formatting branch November 17, 2025 20:00
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