Skip to content

🚀 Feature: Hydration script should run ESLint autofixer #452

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

Closed
3 tasks done
JoshuaKGoldberg opened this issue May 20, 2023 · 4 comments · Fixed by #513 or #514
Closed
3 tasks done

🚀 Feature: Hydration script should run ESLint autofixer #452

JoshuaKGoldberg opened this issue May 20, 2023 · 4 comments · Fixed by #513 or #514
Assignees
Labels
status: accepting prs Please, send a pull request to resolve this! type: feature New enhancement or request

Comments

@JoshuaKGoldberg
Copy link
Owner

Bug Report Checklist

  • I have tried restarting my IDE and the issue persists.
  • I have pulled the latest main branch of the repository.
  • I have searched for related issues and found none that matched my issue.

Overview

#441 added a hydration script that updates an existing repository to use this template's goodness. But the existing repository may have some lint failures at the end of the script, such as using extension-less imports (... from "./file" instead of ... from "./file.js").

Let's have the hydration script run pnpm run lint --fix after it finishes pnpm run format:write!

Additional Info

No response

@JoshuaKGoldberg JoshuaKGoldberg added type: feature New enhancement or request status: accepting prs Please, send a pull request to resolve this! labels May 20, 2023
@JoshuaKGoldberg JoshuaKGoldberg self-assigned this Jun 6, 2023
JoshuaKGoldberg added a commit that referenced this issue Jun 6, 2023
## PR Checklist

- [x] Addresses an existing open issue: fixes #452; fixes #512
- [x] That issue was marked as [`status: accepting
prs`](https://github.com/JoshuaKGoldberg/template-typescript-node-package/issues?q=is%3Aopen+is%3Aissue+label%3A%22status%3A+accepting+prs%22)
- [x] Steps in
[CONTRIBUTING.md](https://github.com/JoshuaKGoldberg/template-typescript-node-package/blob/main/.github/CONTRIBUTING.md)
were taken

## Overview

Adds some more globs to the list of those to remove.
@github-actions
Copy link

github-actions bot commented Jun 6, 2023

🎉 This is included in version v1.28.7 🎉

The release is available on:

Cheers! 📦🚀

1 similar comment
@github-actions
Copy link

github-actions bot commented Jun 6, 2023

🎉 This is included in version v1.28.7 🎉

The release is available on:

Cheers! 📦🚀

JoshuaKGoldberg added a commit that referenced this issue Jun 7, 2023
## PR Checklist

- [x] Addresses an existing open issue: fixes #452
- [x] That issue was marked as [`status: accepting
prs`](https://github.com/JoshuaKGoldberg/template-typescript-node-package/issues?q=is%3Aopen+is%3Aissue+label%3A%22status%3A+accepting+prs%22)
- [x] Steps in
[CONTRIBUTING.md](https://github.com/JoshuaKGoldberg/template-typescript-node-package/blob/main/.github/CONTRIBUTING.md)
were taken

## 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.
@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
status: accepting prs Please, send a pull request to resolve this! type: feature New enhancement or request
Projects
None yet
1 participant