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

Migrate from circleci to github actions #217

Merged
merged 7 commits into from
Sep 16, 2024
Merged

Migrate from circleci to github actions #217

merged 7 commits into from
Sep 16, 2024

Conversation

thodkatz
Copy link
Collaborator

@thodkatz thodkatz commented Sep 4, 2024

No description provided.

@thodkatz thodkatz marked this pull request as draft September 4, 2024 09:09
@thodkatz thodkatz force-pushed the ci-multiple-os branch 27 times, most recently from ceb85a6 to 8f2fbc4 Compare September 4, 2024 15:45
@thodkatz thodkatz force-pushed the ci-multiple-os branch 11 times, most recently from 969cc47 to 85feeac Compare September 12, 2024 18:06
@thodkatz
Copy link
Collaborator Author

thodkatz commented Sep 12, 2024

I think finally it is good to go. The macos and windows workflows fail due to the pickle issue mentioned and resolved here #212. We can either merge the #212 and after this or the reverse :P

I have spoted and resolved another issue (54b4c96) similar to the #218, where if a test fails the process isn't closed.

I have deleted the circlie ci config as well, we can unlink the repo with the circle ci...

@thodkatz thodkatz marked this pull request as ready for review September 12, 2024 18:12
Copy link
Collaborator

@k-dominik k-dominik left a comment

Choose a reason for hiding this comment

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

Hi @thodkatz,

I think this looks great over all, small changes would be appreciated though. And I think it's fine to merge this one first, rebase #212 on the new main and fix tests there.

setup.py Outdated Show resolved Hide resolved
conda-recipe/meta.yaml Outdated Show resolved Hide resolved
tiktorch/rpc/types.py Show resolved Hide resolved
environment.yml Outdated Show resolved Hide resolved
@thodkatz thodkatz force-pushed the ci-multiple-os branch 2 times, most recently from 34544e2 to b2672a4 Compare September 16, 2024 09:30
@thodkatz thodkatz force-pushed the ci-multiple-os branch 4 times, most recently from 37d7a22 to 9a75415 Compare September 16, 2024 12:27
@thodkatz
Copy link
Collaborator Author

thodkatz commented Sep 16, 2024

Okey, I think I have addressed the comments, and the ci is failing only for the pickle issue :) Let me know @k-dominik, and thank you very much!

Copy link
Collaborator

@k-dominik k-dominik left a comment

Choose a reason for hiding this comment

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

lgtm

@thodkatz thodkatz merged commit 7dc4196 into main Sep 16, 2024
6 of 9 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.

2 participants