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

[Build] Output JSCS reports #1329

Merged
merged 3 commits into from
Nov 17, 2016
Merged

[Build] Output JSCS reports #1329

merged 3 commits into from
Nov 17, 2016

Conversation

VWoeltjen
Copy link
Contributor

Records any style violations to dist/reports/checkstyle. Note that jscs-html-reporter doesn't allow file path/name to be specified, so an explicit move is triggered after the reporting step is finished.

Author Checklist

  1. Changes address original issue? Y
  2. Unit tests included and/or updated with changes? N/A
  3. Command line build passes? Y
  4. Changes have been smoke-tested? Y

@VWoeltjen VWoeltjen added this to the Roddenberry milestone Nov 16, 2016
@larkin
Copy link
Contributor

larkin commented Nov 17, 2016

Reviewer Checklist

  1. Changes appear to address issue? Y
  2. Appropriate unit tests included? N/A, build only change.
  3. Code style and in-line documentation are appropriate? Y
  4. Commit messages meet standards? Y

@larkin larkin merged commit c98286d into master Nov 17, 2016
@larkin larkin deleted the jscs-reports-1314 branch November 17, 2016 19:08
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.

2 participants