Skip to content
This repository has been archived by the owner on Jul 30, 2018. It is now read-only.

Add Prettier with tslint plugin and precommit hook #374

Merged
merged 8 commits into from
Jan 2, 2018

Conversation

agubler
Copy link
Member

@agubler agubler commented Dec 17, 2017

Type: feature

The following has been addressed in the PR:

  • There is a related issue
  • All code matches the style guide
  • Unit or Functional tests are included in the PR

Description:

Adds support for Prettier to for code style rules and formatting. Include precommit hook that runs prettier on all .ts files and an npm script that runs prettier against the codebase.

Related to dojo/meta#206

@dylans dylans added this to the beta.5 milestone Dec 22, 2017
M,
T,
O
> {
Copy link
Contributor

Choose a reason for hiding this comment

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

This is kind of ugly.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah agreed there are some cases where it's not always actually "prettier" :)

data.append(decodeURIComponent(name), decodeURIComponent(value));
}
});
text
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did prettier break this out into separate lines but left line 153 alone?

Copy link
Contributor

@edhager edhager left a comment

Choose a reason for hiding this comment

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

I like prettier's opinions on code formatting. :)

I commented on a couple things that seem weird but I don't expect them to be fixed.

@agubler
Copy link
Member Author

agubler commented Jan 2, 2018

Thanks @edhager

@agubler agubler merged commit 7b0fd74 into dojo:master Jan 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants