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

Make ErrorType a RustEnum, and flag it as must_use #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EvanCarroll
Copy link
Member

#5

@lu-zero
Copy link
Member

lu-zero commented Jun 3, 2023

Probably we should setup a CI, are you sure this PR works as it is?

@EvanCarroll
Copy link
Member Author

EvanCarroll commented Jun 3, 2023

@lu-zero The test worked. ;) I'm happy to set up CI, if we're going down that route though -- we should probably move to GitLab too. As that's where the upstream project is and the CI on GitLab is way better. I can set up that up if you'd like.

@lu-zero
Copy link
Member

lu-zero commented Jun 4, 2023

We have a twin organization on gitlab for this purpose, I didn't setup a syncronizer yet, but if you like I'll add you there as well.

@EvanCarroll
Copy link
Member Author

@lu-zero your call! =) When you say "twin organization" i take it to mean you're already set up with GitLab and interested in the migration from GitHub to GitLab. If you want me to handle that, happy to. Alternatively, if you want to keep the projects on GitHub I can look into CI here too, but I have had a lot more experience with GitLab.

@lu-zero
Copy link
Member

lu-zero commented Jun 5, 2023

We have also an org on Gitlab and my plan is to keep both so who prefers gitlab can keep working on gitlab (e.g. @tmatth is developing a qoi impl there) or github and we hopefully can sync the trees as needed.

Since you prefer using gitlab I'm fine with moving the development of svt-av1-rs there, I invited you already there as well. :)

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