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

Feature/1048 Build workflows #1049

Merged
merged 26 commits into from
Nov 19, 2019
Merged

Feature/1048 Build workflows #1049

merged 26 commits into from
Nov 19, 2019

Conversation

klemmchr
Copy link
Member

@klemmchr klemmchr commented Nov 3, 2019

Resolves #1048
Added workflow for build.

In order to get coveralls working, according to their docs there need to be specified 2 secrets:

  • COVERALLS_SERVICE_NAME (the name of your build system)
  • COVERALLS_REPO_TOKEN (the secret repo token from coveralls.io)

In order to allow publishing to npm, there needs to be another secret (e.g. NPM_TOKEN) containing an api key with according permissions.

@klemmchr klemmchr requested a review from knsv November 3, 2019 14:34
@klemmchr klemmchr changed the base branch from master to develop November 3, 2019 14:34
@klemmchr
Copy link
Member Author

klemmchr commented Nov 3, 2019

GitHub Actions is quite faster than travis. 4-5 min vs 8 min. However, currently without percy. Wondering if that matters.

@klemmchr
Copy link
Member Author

klemmchr commented Nov 5, 2019

So, it turns out the release workflow will need some more work to get it done properly. For now I would like to get the build pipelines with GitHub Actions running. Then I can draft the release workflow while we can test the build action.

@klemmchr klemmchr marked this pull request as ready for review November 5, 2019 21:36
@klemmchr
Copy link
Member Author

Want to implement caching afterwards this can be merged.

@klemmchr
Copy link
Member Author

klemmchr commented Nov 17, 2019

Caching is now build in. Build times are reduced to approx. 4 minutes. Also updated jest and it's dependencies to the latest versions.

Copy link
Collaborator

@knsv knsv left a comment

Choose a reason for hiding this comment

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

I think jest-environment-puppeteer, jest-image-snapshot and jest-puppeteer could be removed

@klemmchr
Copy link
Member Author

Will remove them and have a look what happens.

@knsv knsv merged commit 06cb09c into develop Nov 19, 2019
@klemmchr klemmchr deleted the feature/1048_build_workflows branch November 19, 2019 19:12
@klemmchr klemmchr mentioned this pull request Nov 19, 2019
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