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

Updated eslint prettier config so they work better together #1328

Merged
merged 3 commits into from
Jun 27, 2022

Conversation

joshsadam
Copy link
Contributor

Description of changes

See https://prettier.io/docs/en/integrating-with-linters.html

Related issue

Link to the GitHub issue this pull request addresses using the #issuenum format. If it completes an issue, use Fixes #issuenum to automatically close the issue.

Checklist

Things for the developer to confirm they've done before the PR should be accepted:

  • CHANGELOG.md (and UPGRADING.md if necessary) updated with information for new change.
  • Tests added (or description of how to test) for any new features.
  • User documentation updated for UI or technical changes.

@joshsadam joshsadam requested a review from ericenns June 27, 2022 02:13
@joshsadam joshsadam added UI User Interface Issue developer Issues effecting developers of IRIDA, but that aren't user facing labels Jun 27, 2022
Copy link
Member

@ericenns ericenns left a comment

Choose a reason for hiding this comment

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

👍 This looks good to me

@ericenns ericenns merged commit 46ef0b0 into development Jun 27, 2022
@ericenns ericenns deleted the eslint-plugin-prettier branch June 27, 2022 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developer Issues effecting developers of IRIDA, but that aren't user facing UI User Interface Issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants