Skip to content

🚀 Feature: Use eslint-plugin-yml's standard config over recommended #2209

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.

Sign up for GitHub

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

Jump to bottom
Closed
2 tasks done
JoshuaKGoldberg opened this issue Apr 15, 2025 · 1 comment · Fixed by #2210
Closed
2 tasks done

🚀 Feature: Use eslint-plugin-yml's standard config over recommended #2209

JoshuaKGoldberg opened this issue Apr 15, 2025 · 1 comment · Fixed by #2210
Assignees
@JoshuaKGoldberg
Labels
status: accepting prs Please, send a pull request to resolve this! type: feature New enhancement or request

Comments

@JoshuaKGoldberg
Copy link
Owner

Feature Request Checklist

Overview

This repo's eslint.config.js & blockESLintYML.ts set up eslint-plugin-yml with:

https://github.com/bingo-examples/created-typescript-app-everything/blob/ac6c526317195f1e4563dbefc85d836e1f56998a/eslint.config.js#L74

But, the plugin has a "standard" config that enables a few more rules than just "recommended": https://ota-meshi.github.io/eslint-plugin-yml/user-guide/#new-config-eslint-config-js / https://ota-meshi.github.io/eslint-plugin-yml/rules. I'd like to onboard to that.

Additional Info

This is separate from #2208. That issue is about simplifying the config's rules object, and the three currently explicitly enabled rules aren't in the "standard" config.

🎁
@JoshuaKGoldberg JoshuaKGoldberg added status: accepting prs Please, send a pull request to resolve this! type: feature New enhancement or request labels Apr 15, 2025
@JoshuaKGoldberg JoshuaKGoldberg self-assigned this Apr 15, 2025
@JoshuaKGoldberg JoshuaKGoldberg mentioned this issue Apr 15, 2025
Merged
3 tasks
JoshuaKGoldberg added a commit that referenced this issue Apr 15, 2025
@JoshuaKGoldberg
feat: use eslint-plugin-yml's 'standard' config (#2210)
35b86c8 
## PR Checklist

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

## Overview

Swaps over as described by the issue. 

🎁
@github-actions GitHub Actions
Copy link

🎉 This is included in version v2.41.0 🎉

The release is available on:

Cheers! 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JoshuaKGoldberg JoshuaKGoldberg

Labels
status: accepting prs Please, send a pull request to resolve this! type: feature New enhancement or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: use eslint-plugin-yml's 'standard' config JoshuaKGoldberg/create-typescript-app
1 participant
@JoshuaKGoldberg