Skip to content

feat: run ESLint and Prettier fixers in hydration #514

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

Merged
merged 2 commits into from
Jun 7, 2023

Conversation

JoshuaKGoldberg
Copy link
Owner

PR Checklist

Overview

Per the issue, runs pnpm lint --fix near the end of hydration. Also runs pnpm format --write. For both, if there was an error, the user is warned to run the command themselves.

Rewords from "Succeeded" to "Passed" for spinner blocks to indicate that there might not have been a total success.

@codecov
Copy link

codecov bot commented Jun 7, 2023

Codecov Report

Merging #514 (3019539) into main (0bdd69f) will increase coverage by 0.23%.
The diff coverage is 94.44%.

@@            Coverage Diff             @@
##             main     #514      +/-   ##
==========================================
+ Coverage   86.86%   87.10%   +0.23%     
==========================================
  Files          54       55       +1     
  Lines        2901     2947      +46     
  Branches      117      121       +4     
==========================================
+ Hits         2520     2567      +47     
+ Misses        378      377       -1     
  Partials        3        3              
Impacted Files Coverage Δ
src/shared/defaults.ts 59.45% <50.00%> (+0.63%) ⬆️
src/hydrate/hydrate.ts 77.33% <100.00%> (+1.27%) ⬆️
src/hydrate/steps/runCommand.ts 100.00% <100.00%> (ø)
src/shared/cli/spinners.ts 90.69% <100.00%> (+4.65%) ⬆️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@@ -9,7 +11,8 @@ export async function getDefaultSettings() {
.pipeStdout?.($({ stdin: "pipe" })`grep origin`)
.pipeStdout?.($({ stdin: "pipe" })`grep fetch`);
} catch {
console.log(
logLine();
logLine(
Copy link
Owner Author

Choose a reason for hiding this comment

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

Couldn't resist this little cleanup 😄

@JoshuaKGoldberg JoshuaKGoldberg merged commit ac286ac into main Jun 7, 2023
@JoshuaKGoldberg JoshuaKGoldberg deleted the hydration-eslint-autofix branch June 7, 2023 05:45
@github-actions
Copy link

github-actions bot commented Jun 7, 2023

🎉 This is included in version v1.28.8 🎉

The release is available on:

Cheers! 📦🚀

1 similar comment
@github-actions
Copy link

github-actions bot commented Jun 7, 2023

🎉 This is included in version v1.28.8 🎉

The release is available on:

Cheers! 📦🚀

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.

🚀 Feature: Hydration script should run ESLint autofixer
1 participant