Skip to content
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

Allow to pass config instead of configPath #695

Merged
merged 1 commit into from Sep 22, 2019
Merged

Conversation

sheerun
Copy link
Contributor

@sheerun sheerun commented Aug 28, 2019

It would be very useful if someone wants to use lint-staged as library (me)

@codecov
Copy link

codecov bot commented Aug 28, 2019

Codecov Report

Merging #695 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #695   +/-   ##
=======================================
  Coverage   99.67%   99.67%           
=======================================
  Files          11       11           
  Lines         304      304           
  Branches       59       60    +1     
=======================================
  Hits          303      303           
  Misses          1        1
Impacted Files Coverage Δ
src/index.js 100% <100%> (ø) ⬆️

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 8583641...14c46d2. Read the comment docs.

src/index.js Outdated Show resolved Hide resolved
iiroj
iiroj previously approved these changes Sep 2, 2019
@iiroj
Copy link
Member

iiroj commented Sep 2, 2019

Ping @okonet, care to take a look and merge?

okonet
okonet previously requested changes Sep 3, 2019
Copy link
Collaborator

@okonet okonet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please add a documentation of this feature along with an examples how to use to the readme? Thanks!

@iiroj
Copy link
Member

iiroj commented Sep 3, 2019

Good point! We already have a section on the node api, so maybe add the config example there.

@sheerun
Copy link
Contributor Author

sheerun commented Sep 16, 2019

Done

@iiroj iiroj requested a review from okonet September 18, 2019 07:50
@sheerun
Copy link
Contributor Author

sheerun commented Sep 21, 2019

Any chance to merge this?

@iiroj iiroj dismissed okonet’s stale review September 22, 2019 05:25

documentation added

Copy link
Member

@iiroj iiroj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you rebase on master and rename your commits to loosely match the Conventional Commits spec:

  • feat: allow to pass config instead of configPath
  • refactor: resolve config in same format as loadConfig
  • docs: add docs

After this I can merge!

@sheerun sheerun force-pushed the config branch 2 times, most recently from 7ccff7b to bb37c68 Compare September 22, 2019 10:07
@sheerun
Copy link
Contributor Author

sheerun commented Sep 22, 2019

done

@iiroj iiroj merged commit 88d9d4f into lint-staged:master Sep 22, 2019
@okonet
Copy link
Collaborator

okonet commented Sep 22, 2019

🎉 This PR is included in version 9.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

3 participants