Added includedIgnoreFile() function that is exported from @eslint/compat
Added convertIgnorePatternToMinimatch() function that is exported from @eslint/compat (purposely undocumented because I'm not sure how end users would use this)
Updated README for @eslint/compat
Switched @eslint/migrate-config to use the convertIgnorePatternToMinimatch() function from @eslint/compat instead of duplicating that functionality
Rearranged the order in which packages are released so @eslint/migrate-config comes after @eslint/compat due to the new dependency on it
Note: The includeIgnoreFile() function differs from the original issue in that there is no second argument. I'm not sure it's needed and it could complicate the process if the ignore file is in an ancestor directory. For this first release, I'd like to see what kind of feedback we get as-is.
Related Issues
fixes #44
Is there anything you'd like reviewers to focus on?
Prerequisites checklist
What is the purpose of this pull request?
Add a
includeIgnoreFile()
function.What changes did you make? (Give an overview)
includedIgnoreFile()
function that is exported from@eslint/compat
convertIgnorePatternToMinimatch()
function that is exported from@eslint/compat
(purposely undocumented because I'm not sure how end users would use this)@eslint/compat
@eslint/migrate-config
to use theconvertIgnorePatternToMinimatch()
function from@eslint/compat
instead of duplicating that functionality@eslint/migrate-config
comes after@eslint/compat
due to the new dependency on itNote: The
includeIgnoreFile()
function differs from the original issue in that there is no second argument. I'm not sure it's needed and it could complicate the process if the ignore file is in an ancestor directory. For this first release, I'd like to see what kind of feedback we get as-is.Related Issues
fixes #44
Is there anything you'd like reviewers to focus on?