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: Enable rust tests in CI #50

Merged
merged 6 commits into from
Feb 21, 2023
Merged

chore: Enable rust tests in CI #50

merged 6 commits into from
Feb 21, 2023

Conversation

lexnv
Copy link
Contributor

@lexnv lexnv commented Feb 20, 2023

This PR re-enables the github CI that was not running due to

 Invalid workflow file: .github/workflows/rust.yml#L74
The workflow is not valid. .github/workflows/rust.yml (Line: 74, Col: 9): 'uses' is already defined .github/workflows/rust.yml (Line: 81, Col: 9): 'uses' is already defined

While at it, add some extra checks for documentation, style and tests.

// CC: @paritytech/tools-team

Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
@lexnv lexnv self-assigned this Feb 20, 2023
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
profile: minimal
toolchain: stable
components: clippy
override: true
Copy link
Member

Choose a reason for hiding this comment

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

you can add target: wasm32-unknown-unknown here to avoid install it manually below.

Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
@lexnv lexnv merged commit f84ba3f into main Feb 21, 2023
@lexnv lexnv deleted the lexnv/rust_workflow branch February 21, 2023 12:01
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.

4 participants