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

Fix lint-staged config #77

Merged
merged 1 commit into from
Oct 13, 2020
Merged

Fix lint-staged config #77

merged 1 commit into from
Oct 13, 2020

Conversation

simonihmig
Copy link
Member

Tested on kaliber5-website, this should be finally the version that works correctly, running yarn lint:ts on all project files instead of just the staged ones (as it was previously).

Tested on kaliber5-website, this should be finally the version that works correctly, running `yarn lint:ts` on *all*  project files instead of just the staged ones (as it was previously).
@simonihmig simonihmig added the bug Something isn't working label Oct 13, 2020
@simonihmig simonihmig requested a review from lolmaus October 13, 2020 09:53
Copy link
Contributor

@lolmaus lolmaus left a comment

Choose a reason for hiding this comment

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

Hmm. What actual difference does this change make? Does lint-staged pass a file name only when ivoked as a string, i. e. without an arrow function?

@simonihmig simonihmig merged commit 6f84abd into master Oct 13, 2020
@simonihmig simonihmig deleted the fix-lintstaged branch October 13, 2020 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants