Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Use EditorConfig **DRAFT** #26

Draft
wants to merge 3 commits into
base: main-forked
Choose a base branch
from
Draft

Use EditorConfig **DRAFT** #26

wants to merge 3 commits into from

Conversation

jakewan
Copy link
Collaborator

@jakewan jakewan commented Aug 8, 2022 •
edited
Loading

  • Move settings from .prettierrc.js to .editorconfig
  • Update Prettier version
  • Add formatcode npm script
  • Run npm run formatcode

@jakewan
Use .editorconfig
4c707a0 
Includes:

- Update Prettier
- Add `formatcode` npm script
- Run `npm run formatcode`
@jakewan
Add .prettierignore
19c015e
@jakewan
Remove unneeded Prettier settings
2611d8e
@jakewan jakewan added the tech debt Primarily addresses technical debt label Aug 8, 2022
@jakewan jakewan changed the title Use EditorConfig Use EditorConfig **DRAFT** Aug 12, 2022
@jakewan jakewan marked this pull request as draft August 12, 2022 19:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@enmatt enmatt Awaiting requested review from enmatt enmatt is a code owner

@pyang-1981 pyang-1981 Awaiting requested review from pyang-1981 pyang-1981 is a code owner

@LyleScott LyleScott Awaiting requested review from LyleScott LyleScott is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
tech debt Primarily addresses technical debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jakewan