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

chore: add ci github actions #12

Merged
merged 1 commit into from
Mar 30, 2021
Merged

Conversation

vasco-santos
Copy link
Member

#10 added linting errors but there is no CI. This PR fixes both of these

@vasco-santos vasco-santos force-pushed the chore/add-ci-github-actions branch from 497da60 to 7983f0b Compare March 30, 2021 10:33
@vasco-santos vasco-santos requested a review from rvagg March 30, 2021 10:34
Copy link
Member

@alanshaw alanshaw left a comment

Choose a reason for hiding this comment

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

LGTM - lint script is currently run from the test script so it's double linting right now. Assuming that'll be fixed up later...?

@vasco-santos
Copy link
Member Author

vasco-santos commented Mar 30, 2021

Yes, I will create an issue to track adding tests. The linting is really fast, I thing we can keep this already running the test script

EDIT: There is already #3 , just linking it for now

@vasco-santos vasco-santos merged commit 41b5bd8 into master Mar 30, 2021
@vasco-santos vasco-santos deleted the chore/add-ci-github-actions branch March 30, 2021 13:23
github-actions bot pushed a commit that referenced this pull request Aug 20, 2021
## 1.0.0 (2021-08-20)

### Features

* add generated TS declarations ([#7](#7)) ([e52fa9c](e52fa9c))
* add method docs ([#8](#8)) ([4921f7b](4921f7b))
* support namespaces other than Filecoin ([#10](#10)) ([918bdc0](918bdc0))

### Bug Fixes

* add release-it to devDeps ([e885a79](e885a79))
* add storage miner and worker API methods ([#9](#9)) ([086fc98](086fc98))

### Trivial Changes

* add 0.0.13 releae docs ([33da8b7](33da8b7))
* add 0.0.14 changelog ([3bb977e](3bb977e))
* add ci github actions ([#12](#12)) ([41b5bd8](41b5bd8))
* add releasing section to readme ([#13](#13)) ([b23ef5a](b23ef5a))
* auto-releases with semantic-release ([a8b62ce](a8b62ce))
* clean up README ([9ea2966](9ea2966))
* update changelog ([231dc2b](231dc2b))
* update deps ([0216565](0216565))
* update to lotus 1.5.2 ([#11](#11)) ([54330d0](54330d0))
* update type declarations for Lotus v1.1.2 ([b7646fc](b7646fc))
@github-actions
Copy link

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants