Skip to content
This repository was archived by the owner on Dec 2, 2024. It is now read-only.
/ level-js Public archive

Move to GitHub Actions#211

Merged
vweevers merged 5 commits intoLevel:masterfrom
juanpicado:master
Apr 19, 2021
Merged

Move to GitHub Actions#211
vweevers merged 5 commits intoLevel:masterfrom
juanpicado:master

Conversation

@juanpicado
Copy link
Copy Markdown
Contributor

@juanpicado juanpicado commented Apr 19, 2021

Ref Level/community#99

Currently are failing due Sauce Labs credentials are required error on my repo.

@vweevers vweevers mentioned this pull request Apr 19, 2021
60 tasks
@vweevers vweevers self-requested a review April 19, 2021 17:40
Co-authored-by: Vincent Weevers <mail@vincentweevers.nl>
@vweevers
Copy link
Copy Markdown
Member

We're almost there 😃 To make coverage reports work, can you update the coverage npm script and remove coveralls from devDependencies? See the similar change to package.json in Level/concat-iterator@a229bb4#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519

Copy link
Copy Markdown
Member

@vweevers vweevers left a comment

Choose a reason for hiding this comment

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

Amazing, thank you! I will shortly create a branch (that has access to secrets) so we can see it in action (pun intended).

@juanpicado
Copy link
Copy Markdown
Contributor Author

Amazing, thank you! I will shortly create a branch (that has access to secrets) so we can see it in action (pun intended).

super thanks :) I'll back tomorrow to convert one more.

@vweevers
Copy link
Copy Markdown
Member

Tests passed and coverage works: https://app.codecov.io/gh/Level/level-js

@vweevers vweevers merged commit a991f3b into Level:master Apr 19, 2021
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