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

Use EditorConfig **DRAFT** #26

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

Use EditorConfig **DRAFT** #26

jakewan 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 added 3 commits August 8, 2022 09:58
@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.

2 participants

@jakewan