Skip to content

Conversation

@suyask-msft
Copy link
Collaborator

This pull request makes a minor update to the Python workflow configuration. The change removes the continue-on-error: true setting from the "Check format with black" job, so formatting errors will now cause the workflow to fail instead of being ignored.

suyask-msft and others added 6 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
@suyask-msft suyask-msft requested a review from a team as a code owner November 17, 2025 19:17
Copilot AI review requested due to automatic review settings November 17, 2025 19:17
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 Python code formatting standards by removing the continue-on-error: true flag from the black formatting check in the GitHub Actions workflow. This change ensures that CI builds will fail if code doesn't meet the project's formatting standards, rather than just issuing warnings.

Key Changes:

  • Removed continue-on-error: true setting from the "Check format with black" workflow step
  • Removed the associated TODO comment indicating formatting errors needed to be fixed

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

@suyask-msft suyask-msft merged commit b454f67 into main Nov 17, 2025
14 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.

3 participants