Skip to content

feat: add new rule no-array-fill-with-reference-type without fix solution #2661

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
Open
wants to merge 58 commits into
base: main
Choose a base branch
from
Open

feat: add new rule no-array-fill-with-reference-type without fix solution #2661

wants to merge 58 commits into from

Conversation

legend80s
Copy link

liuchuanzong and others added 30 commits May 29, 2025 09:04
docs: add tips of how to install for new developers
87f000c
fix: error when npm run create-rule > Cannot find package 'lodash-es'…
a8f6d3f 
… imported from ...create-rule.js
feat: add new rule no-array-fill-with-reference-type without fix solu…
ea97dd1 
…tion
feat(no-array-fill-with-reference-type): only check fill on builtin A…
35057ee 
…rray
test(no-array-fill-with-reference-type): add more valid cases
aded9b0
docs(no-array-fill-with-reference-type): remove comments
57f4d74
docs(no-array-fill-with-reference-type): gen code by npm run fix:esli…
85bb03a 
…nt-docs using Node.js v22.16.0
docs(no-array-fill-with-reference-type): brief the desc and recover r…
3977224 
…eadme.md to main generated by npm run fix:eslint-docs using Node.js v22.16.0
docs: no file configs/recommended.js
8243f14
docs(no-array-fill-with-reference-type): gen code by npm run fix:esli…
3635dd3 
…nt-docs using Node.js v22.16.0
@legend80s
docs: add tips of how to install for new developers
fcd690e
@legend80s
fix: error when npm run create-rule > Cannot find package 'lodash-es'…
51af337 
… imported from ...create-rule.js
@legend80s
feat: add new rule no-array-fill-with-reference-type without fix solu…
09ca3b5 
…tion
@legend80s
feat(no-array-fill-with-reference-type): only check fill on builtin A…
b20ddf2 
…rray
@legend80s
test(no-array-fill-with-reference-type): add more valid cases
c70d17f
@legend80s
docs(no-array-fill-with-reference-type): remove comments
7c7fa38
@legend80s
docs(no-array-fill-with-reference-type): gen code by npm run fix:esli…
d3a4179 
…nt-docs using Node.js v22.16.0
@legend80s
docs(no-array-fill-with-reference-type): brief the desc and recover r…
681827d 
…eadme.md to main generated by npm run fix:eslint-docs using Node.js v22.16.0
@legend80s
docs: no file configs/recommended.js
a6ca789
@legend80s
docs(no-array-fill-with-reference-type): gen code by npm run fix:esli…
2091479 
…nt-docs using Node.js v22.16.0
@legend80s
Merge branch 'main' of github.com:legend80s/eslint-plugin-unicorn
d80842c
@legend80s
feat(no-array-fill-with-reference-type): no fixable
90b768b
@legend80s
docs(no-array-fill-with-reference-type): no fixable
a0841ad
@legend80s
feat(no-array-fill-with-reference-type): check Array.from().fill
ab37dcb
@legend80s
docs(no-array-fill-with-reference-type): correct docs
6b14629
@legend80s
docs(no-array-fill-with-reference-type): correct docs by npm run fix:…
b6bf56c 
…eslint-docs
@legend80s
test: disable no-array-fill-with-reference-type because it is safe du…
928e51a 
…e to no mutation to `errors`
@legend80s
feat: hide unknown type in error message
36b04e3
@legend80s
test: hide unknown type in error message
4f4cf65
@legend80s
test: add a case of hiding unknown type in error message
3813e76
fisker
fisker reviewed Jun 12, 2025
View reviewed changes
fisker
fisker reviewed Jun 12, 2025
View reviewed changes
legend80s added 8 commits June 13, 2025 16:35
fisker
fisker reviewed Jun 27, 2025
View reviewed changes
fisker
fisker reviewed Jun 27, 2025
View reviewed changes
fisker
fisker reviewed Jun 27, 2025
View reviewed changes
fisker
fisker reviewed Jul 15, 2025
View reviewed changes
fisker
fisker reviewed Jul 15, 2025
View reviewed changes
fisker
fisker reviewed Jul 15, 2025
View reviewed changes
@sindresorhus
Copy link
Owner

Can you fix the merge conflicts?

@legend80s
Copy link
Author

Can you fix the merge conflicts?

OK

@legend80s
Copy link
Author

@fisker Could you check the CI failures? They don’t appear to be related to this PR.

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

Reviewers

@fisker fisker fisker left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@legend80s @sindresorhus @fisker