Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: ignored file #266

Merged
merged 1 commit into from
Mar 21, 2022
Merged

fix: ignored file #266

merged 1 commit into from
Mar 21, 2022

Conversation

ricardogobbosouza
Copy link
Collaborator

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

Files ignored in .stylelintignore when changed were going through lint

Resolve #257

Breaking Changes

No

Additional Info

Stylelint has an api to check if the file is ignored isPathIgnored

@codecov
Copy link

codecov bot commented Mar 17, 2022

Codecov Report

Merging #266 (7d6b8c9) into master (79fb44d) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #266   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines          219       225    +6     
  Branches        51        51           
=========================================
+ Hits           219       225    +6     
Impacted Files Coverage Δ
src/getStylelint.js 100.00% <100.00%> (ø)
src/index.js 100.00% <100.00%> (ø)
src/linter.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 79fb44d...7d6b8c9. Read the comment docs.

@ricardogobbosouza ricardogobbosouza merged commit ac50bf5 into master Mar 21, 2022
@ricardogobbosouza ricardogobbosouza deleted the fix-ignore-file branch March 21, 2022 12:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants