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

Add a basic CI workflow to catch test/doc/compile failures #40

Merged
merged 6 commits into from
Jun 15, 2021

Conversation

jsdw
Copy link
Contributor

@jsdw jsdw commented Jun 14, 2021

A basic workflow adapted from jsonrpsee to run basic compile/test/doc checks for us.

@jsdw jsdw mentioned this pull request Jun 14, 2021
Copy link
Member

@niklasad1 niklasad1 left a comment

Choose a reason for hiding this comment

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

LGTM, maybe we should enable clippy too but could be another PR

I suppose the CI doesn't run because of branch protection.

@jsdw
Copy link
Contributor Author

jsdw commented Jun 15, 2021

LGTM, maybe we should enable clippy too but could be another PR

I suppose the CI doesn't run because of branch protection.

Definitely; I contemplated adding clippy and rustfmt too but was hoping to get this PR into the release to simplify the checklist, so my initial instinct was to avoid anything that would require code changes too.

And yup, @dvdplm can admin the repo now so we should be able to sort out that blocking check :)

@dvdplm dvdplm merged commit 3bdc666 into develop Jun 15, 2021
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.

3 participants