Skip to content
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.

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

refactor(oxlint): move ConfigFileInit output outside CliRunResult, exit code 1 when it fails #8776

Conversation

Sysix
Copy link
Collaborator

@Sysix Sysix commented Jan 29, 2025

No description provided.

Copy link
Collaborator Author

Sysix commented Jan 29, 2025


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • 0-merge - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions github-actions bot added the A-cli Area - CLI label Jan 29, 2025
@github-actions github-actions bot added the C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior label Jan 29, 2025
@Sysix Sysix force-pushed the 01-29-refactor_oxlint_move_configfileinit_output_outside_clirunresult_exit_code_1_when_it_fails branch from 7c47149 to 36acc12 Compare January 29, 2025 18:16
@Sysix Sysix marked this pull request as ready for review January 29, 2025 19:13
@Sysix Sysix force-pushed the 01-29-refactor_oxlint_move_configfileinit_output_outside_clirunresult_exit_code_1_when_it_fails branch from 36acc12 to 9b5ba43 Compare January 29, 2025 19:41
@graphite-app graphite-app bot added the 0-merge Merge with Graphite Merge Queue label Jan 30, 2025
Copy link

graphite-app bot commented Jan 30, 2025

Merge activity

@Boshen Boshen force-pushed the 01-29-refactor_oxlint_remove_unused_clirunresult_pathnotfound_ branch from f952b61 to f4cecb5 Compare January 30, 2025 01:43
@Boshen Boshen force-pushed the 01-29-refactor_oxlint_move_configfileinit_output_outside_clirunresult_exit_code_1_when_it_fails branch from 9b5ba43 to 2378fef Compare January 30, 2025 01:44
Base automatically changed from 01-29-refactor_oxlint_remove_unused_clirunresult_pathnotfound_ to main January 30, 2025 01:48
@graphite-app graphite-app bot merged commit 2378fef into main Jan 30, 2025
16 checks passed
@graphite-app graphite-app bot deleted the 01-29-refactor_oxlint_move_configfileinit_output_outside_clirunresult_exit_code_1_when_it_fails branch January 30, 2025 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0-merge Merge with Graphite Merge Queue A-cli Area - CLI C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant