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

The "Pretty" PR #1305

Merged
merged 9 commits into from Mar 24, 2021
Merged

The "Pretty" PR #1305

merged 9 commits into from Mar 24, 2021

Conversation

Copy link
Contributor

@marimeireles marimeireles commented Mar 24, 2021

Adds configuration for both prettier and eslint. Enforces prettier rules but not eslint rules.

@marimeireles marimeireles changed the title WIP - The "Pretty" PR The "Pretty" PR Mar 24, 2021
@@ -62,6 +63,9 @@ jobs:
yarn install
yarn run test
- name: Run prettyfier
run: yarn run prettier --check .
Copy link
Member

@martinRenou martinRenou Mar 24, 2021

Choose a reason for hiding this comment

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

run: |
  cd js
  yarn run prettier --check .

Copy link
Contributor Author

@marimeireles marimeireles Mar 24, 2021

Choose a reason for hiding this comment

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

Ah, I thought it'd pick it up from the command above.
Thanks!

Copy link
Member

@martinRenou martinRenou Mar 24, 2021

Choose a reason for hiding this comment

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

I recall there was a working-directory option that you can use otherwise. Something like:

run: yarn run prettier --check .
working-directory: js

@martinRenou martinRenou merged commit 211af8f into bqplot:master Mar 24, 2021
8 checks passed
martinRenou pushed a commit to martinRenou/bqscales that referenced this issue Aug 19, 2021
martinRenou pushed a commit to martinRenou/bqscales that referenced this issue Aug 19, 2021
martinRenou pushed a commit to martinRenou/bqscales that referenced this issue Aug 19, 2021
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.

None yet

2 participants