Skip to content

Conversation

@deborahbrouwer
Copy link
Contributor

Now that Tyr is upstream, the process for contributing has changed. Update the documentation about the development process to help new contributors.

Also, while here, update progress in the downstream driver.

Now that Tyr is upstream, the process for contributing has changed.
Update the documentation about the development process to help
new contributors.

Also, while here, update progress in the downstream driver.

Signed-off-by: Deborah Brouwer <deborah.brouwer@collabora.com>
Copy link
Member

@ojeda ojeda left a comment

Choose a reason for hiding this comment

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

Some suggestions, but up to @dwlsalmeida of course.

Co-authored-by: Miguel Ojeda <ojeda@users.noreply.github.com>
@dwlsalmeida
Copy link

@deborahbrouwer

Hi Deb,

Please update the IGT link. Our IGT code is now upstream as well, you don't need a special branch for it. If you could instead link to the IGT directory where our tests live, that would be better. Perhaps link to their CONTRIBUTORS.md file if you think it's a good idea.

Other than that, LGTM.

Signed-off-by: Deborah Brouwer <deborah.brouwer@collabora.com>
@deborahbrouwer
Copy link
Contributor Author

Please update the IGT link. Our IGT code is now upstream as well, you don't need a special branch for it. If you could instead link to the IGT directory where our tests live, that would be better. Perhaps link to their CONTRIBUTORS.md file if you think it's a good idea.

Thanks @dwlsalmeida I updated the IGT repo link, but didn't link to the IGT contributors page to avoid mixing the messages with contributing to Tyr.

@ojeda I think this should be ready to go now unless you have any more concerns/comments?

@ojeda ojeda merged commit 47f4b7f into Rust-for-Linux:main Dec 3, 2025
ojeda added a commit that referenced this pull request Dec 3, 2025
@ojeda
Copy link
Member

ojeda commented Dec 3, 2025

I recreated the merge to squash the commits, I hope that is fine with you. It is live now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants