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

fix(ci): Detect TS errors and check code quality & formatting in CI #430

Merged
merged 9 commits into from
Jan 23, 2021

Conversation

adrienjoly
Copy link
Member

@adrienjoly adrienjoly commented Jan 17, 2021

To detect TypeScript errors and prevent regressions of code quality in Pull Requests / CI.

  • fix linter issues
  • also run npx tsc --noEmit on CI

@adrienjoly adrienjoly self-assigned this Jan 17, 2021
=> to prevent eslint from failing (for now)

/Users/adrienjoly/dev/openwhyd/openwhyd/app/snip.js
  800:5  error  Variables should not be deleted  no-delete-var
…into fix/ci-linter

* 'fix/ci-linter' of https://github.com/openwhyd/openwhyd:
  chore(release): 1.44.22 [skip ci]
  fix(ci): Revert Applitools configuration for GitHub integration (#436)
  chore(release): 1.44.21 [skip ci]
  fix(ci): Add APPLITOOLS_BATCH_ID env var (#432)
  chore(release): 1.44.20 [skip ci]
  fix(UI): Search results blocked by header, when visiting the home page (#435)
  chore(release): 1.44.19 [skip ci]
  fix(ci): Don't npm install unless we publish to Docker Hub (#431)
  chore(release): 1.44.18 [skip ci]
  fix(deps): Update mongodb 3.1.13 --> 3.6.3 (#427)
@adrienjoly
Copy link
Member Author

Results of execution in GitHub Actions CI:

image

@adrienjoly adrienjoly marked this pull request as ready for review January 23, 2021 14:53
@adrienjoly adrienjoly changed the title fix(ci): Run linter fix(ci): Detect TS errors and check code quality & formatting in CI Jan 23, 2021
@adrienjoly adrienjoly merged commit c4b4dee into master Jan 23, 2021
@adrienjoly adrienjoly deleted the fix/ci-linter branch January 23, 2021 14:59
adrienjoly pushed a commit that referenced this pull request Jan 23, 2021
## [1.44.23](v1.44.22...v1.44.23) (2021-01-23)

### Bug Fixes

* **ci:** Detect TS errors and check code quality & formatting in CI ([#430](#430)) ([c4b4dee](c4b4dee))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant