Skip to content

feat(scripts): use lint-staged to format #865

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

Merged
merged 1 commit into from
Jul 25, 2022
Merged

Conversation

shortcuts
Copy link
Member

🧭 What and Why

🎟 JIRA Ticket: https://algolia.atlassian.net/browse/APIC-593

Changes included:

Follow up of #856

Add other format to the pre-commit hook.

We can leverage lint-staged to do those operations so I've removed the custom logic.

🧪 Test

Change any of the files under the .lintstagedrc config and see the linters running

@shortcuts shortcuts self-assigned this Jul 25, 2022
@netlify
Copy link

netlify bot commented Jul 25, 2022

Deploy Preview for api-clients-automation canceled.

Name Link
🔨 Latest commit 9cfc823
🔍 Latest deploy log https://app.netlify.com/sites/api-clients-automation/deploys/62de72055b19720008bafa92

@algolia-bot
Copy link
Collaborator

algolia-bot commented Jul 25, 2022

✗ The generated branch has been deleted.

If the PR has been merged, you can check the generated code on the main branch.
You can still access the code generated on main via this commit.

@shortcuts shortcuts requested a review from bodinsamuel July 25, 2022 10:53
@shortcuts shortcuts merged commit 5d08b79 into main Jul 25, 2022
@shortcuts shortcuts deleted the feat/scripts-lint-staged branch July 25, 2022 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants