Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Added linter in CONTRIBUTING.md #870

Merged
merged 2 commits into from
Dec 15, 2016
Merged

Added linter in CONTRIBUTING.md #870

merged 2 commits into from
Dec 15, 2016

Conversation

jajodiaraghav
Copy link
Contributor

Fixes #375.

Changes proposed in this pull request:

  • The CONTRIBUTING.md has been updated with the description and usage of "Linters"
  • Links to linter packages for common text editors has also been added.

Thanks!

cc @HospitalRun/core-maintainers

Copy link
Member

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @jajodiaraghav!

We should call out the specific linters used by HospitalRun instead of generic linters for various editors.
HospitalRun uses the following linters:
ESLint and stylelint

Copy link
Member

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks for updating the PR @jajodiaraghav!

@jkleinsc jkleinsc merged commit d05d753 into HospitalRun:master Dec 15, 2016
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.

2 participants