Skip to content

[BE]Enhance CI workflow error messages for formatting and linting checks#34

Closed
FindHao wants to merge 1 commit intomainfrom
findhao/add_format_message
Closed

[BE]Enhance CI workflow error messages for formatting and linting checks#34
FindHao wants to merge 1 commit intomainfrom
findhao/add_format_message

Conversation

@FindHao
Copy link
Member

@FindHao FindHao commented Jul 15, 2025

Summary:
Updated the GitHub Actions workflow to provide clearer error messages when formatting and linting checks fail. The new messages instruct users to run 'make format' to resolve issues and include a link to the detailed formatting guide.

Key Changes:

  • Improved error handling in the format-check and lint-check jobs to guide users on fixing formatting issues.

This change aims to enhance user experience by providing immediate feedback and resources for resolving code quality issues.

Summary:
Updated the GitHub Actions workflow to provide clearer error messages when formatting and linting checks fail. The new messages instruct users to run 'make format' to resolve issues and include a link to the detailed formatting guide.

Key Changes:
- Improved error handling in the format-check and lint-check jobs to guide users on fixing formatting issues.

This change aims to enhance user experience by providing immediate feedback and resources for resolving code quality issues.
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 15, 2025
@facebook-github-bot
Copy link
Contributor

@FindHao has imported this pull request. If you are a Meta employee, you can view this in D78311326.

@facebook-github-bot
Copy link
Contributor

@FindHao merged this pull request in 8540402.

@FindHao FindHao deleted the findhao/add_format_message branch September 19, 2025 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot. Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants