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

style(stark-all): re-format files with latest Prettier version. Include MD files in Prettier auto formatting. #428

Merged

Conversation

christophercr
Copy link
Collaborator

@christophercr christophercr commented Jun 11, 2018

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[X] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Formatting applied using a previous version of Prettier. MD files were not included in the set of files to be auto-formatted.

What is the new behavior?

Re-format files using the latest version of Prettier. MD files also auto-formatted.

Important: the CHANGELOG.md file is excluded from this auto-formatting because it is auto-generated on every release!

Does this PR introduce a breaking change?

[ ] Yes
[X] No

@coveralls
Copy link

coveralls commented Jun 11, 2018

Coverage Status

Coverage increased (+0.01%) to 94.259% when pulling 2d56514 on christophercr:feature/fix-formatting into 18ddc7a on NationalBankBelgium:master.

@christophercr christophercr force-pushed the feature/fix-formatting branch from 2d56514 to 86d82c7 Compare June 11, 2018 09:18
@christophercr christophercr merged commit 0ced55c into NationalBankBelgium:master Jun 11, 2018
@christophercr christophercr deleted the feature/fix-formatting branch June 11, 2018 10:41
@dsebastien dsebastien added this to the 10.0.0-alpha.3 milestone Jun 11, 2018
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.

3 participants