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

CI Update #204

Merged
17 commits merged into from
May 6, 2021
Merged

CI Update #204

17 commits merged into from
May 6, 2021

Conversation

ghost
Copy link

@ghost ghost commented May 4, 2021

Currently, we are hitting the execution limit of 50 minutes in Travis, so removing the tests and using the coverage for checking the tests as well.

@ghost ghost added the enhancement New feature or request label May 4, 2021
@ghost ghost self-assigned this May 4, 2021
@ghost
Copy link
Author

ghost commented May 4, 2021

Changes Introduced:

  • Original Travis divided, and now only have coverage in it. (The current running time is at max 24 minutes)
  • Github Actions is now being used for tests. (The current running time is at max 13 minutes)
  • README updated to reflect the build status and test coverage.
  • hardhat-docgen added to create technical documentation with inline coding (will perfectly fit with @computerphysicslab 's recent documentation task).
  • Some prettier commands updated (write and check commands added).

@ghost ghost marked this pull request as ready for review May 4, 2021 12:43
@ghost ghost requested a review from ororopickpocket May 4, 2021 12:43
@ghost
Copy link
Author

ghost commented May 4, 2021

IMPORTANT: To speed up the CI, I am using npm ci instead of npm i, which makes use of package-lock.json.

Thus from now onwards, we should install all packages using npm, and not from yarn.

@ghost ghost merged commit ea4e257 into development May 6, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants