Skip to content

Test workflow#104

Merged
sam0x17 merged 2 commits intoopentensor:stagingfrom
igorsyl:feat/tests
Feb 27, 2025
Merged

Test workflow#104
sam0x17 merged 2 commits intoopentensor:stagingfrom
igorsyl:feat/tests

Conversation

@igorsyl
Copy link
Contributor

@igorsyl igorsyl commented Feb 20, 2025

Adds check-rust workflow.
Inspired by #64

@camfairchild
Copy link
Contributor

Yes cargo would be great as well

@ibraheem-abe
Copy link
Contributor

Hey @igorsyl
We already run pytests here: https://github.com/opentensor/btwallet/blob/main/.circleci/config.yml

But the cargo should be a good addition!

@igorsyl
Copy link
Contributor Author

igorsyl commented Feb 20, 2025

I brought over https://github.com/opentensor/subtensor/blob/main/.github/workflows/check-rust.yml
What image does the SubtensorCI runner use so I can test the workflow on my end?
cc @ibraheem-opentensor @camfairchild

@camfairchild
Copy link
Contributor

Cc: @sam0x17

@sam0x17
Copy link
Contributor

sam0x17 commented Feb 21, 2025

I brought over https://github.com/opentensor/subtensor/blob/main/.github/workflows/check-rust.yml What image does the SubtensorCI runner use so I can test the workflow on my end? cc @ibraheem-opentensor @camfairchild

nice, but yeah SubtensorCi is ubuntu-latest based but it has crazy specs

@sam0x17
Copy link
Contributor

sam0x17 commented Feb 21, 2025

and actually, I just did a big overhaul of the check_rust file, would you mind bringing in those changes?

@sam0x17
Copy link
Contributor

sam0x17 commented Feb 21, 2025

see current devnet-ready branch content

@igorsyl
Copy link
Contributor Author

igorsyl commented Feb 21, 2025

@sam0x17 could you approve the workflow run?

@sam0x17
Copy link
Contributor

sam0x17 commented Feb 24, 2025

looks like a merge conflict + compile error now

@igorsyl
Copy link
Contributor Author

igorsyl commented Feb 25, 2025

looks like a merge conflict + compile error now

Fixed new merge conflict. Please approve the workflow. Could you also share the image path or build instructions for the SubtensorCI runner so I can run the workflow myself? @sam0x17

@sam0x17 sam0x17 merged commit b980889 into opentensor:staging Feb 27, 2025
6 of 13 checks passed
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