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

Add prettier to check formatting during CI linting #557

Merged
merged 7 commits into from
Feb 26, 2023
Merged

Conversation

TheDevMinerTV
Copy link
Member

Blocked by #553

@ImUrX
Copy link
Member

ImUrX commented Feb 5, 2023

Why don’t you push the change to #553

@ButterscotchV ButterscotchV added Area: GUI Related to the GUI Area: Continuous Integration Automated testing and deployment labels Feb 5, 2023
@TheDevMinerTV
Copy link
Member Author

Why don’t you push the change to #553

it's in your fork of the repo, not in this one

@ImUrX
Copy link
Member

ImUrX commented Feb 5, 2023

you still have write perms
image

@TheDevMinerTV
Copy link
Member Author

Also, your PR title would be incorrect

Copy link
Member

@Erimelowo Erimelowo left a comment

Choose a reason for hiding this comment

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

lgtm, but why does CI flag an error (can't reproduce locally)?

@Eirenliel
Copy link
Member

Plz fix CI, @TheDevMinerTV

@TheDevMinerTV TheDevMinerTV force-pushed the ci/add-prettier branch 3 times, most recently from ba7202d to 69c0df1 Compare February 12, 2023 18:17
Eirenliel and others added 6 commits February 17, 2023 12:03
Quoting https://prettier.io/docs/en/integrating-with-linters.html:

> The downsides of those plugins are:
> You end up with a lot of red squiggly lines in your editor, which > gets annoying. Prettier is supposed to make you forget about formatting – and not be in your face about it!
> They are slower than running Prettier directly.
> They’re yet one layer of indirection where things may break.
@Eirenliel Eirenliel merged commit 69c175c into main Feb 26, 2023
@Eirenliel Eirenliel deleted the ci/add-prettier branch February 26, 2023 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Continuous Integration Automated testing and deployment Area: GUI Related to the GUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants