Skip to content

Conversation

@sonnyp
Copy link
Contributor

@sonnyp sonnyp commented Dec 16, 2023

Fixes #608 (comment)

This PR

@sonnyp sonnyp requested a review from Hofer-Julian December 16, 2023 13:00
@Hofer-Julian
Copy link
Contributor

I see the PR is still a draft. Should I give it a try anyway?

@sonnyp
Copy link
Contributor Author

sonnyp commented Dec 26, 2023

Yeah early feedback and what you think of this approach would be useful

@Hofer-Julian
Copy link
Contributor

I have troubles reviewing it because of the portal bug.
I tried to give it host filesystem permissions in the manifest, but unfortunately that didn't help.

Copy link
Contributor

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

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

I've tested demos as well as https://github.com/adzialocha/gnome-p2panda-workshop and both work.

The code changes look fine to me as well.
Thanks, @sonnyp!

@sonnyp sonnyp marked this pull request as ready for review January 8, 2024 23:43
@sonnyp sonnyp requested a review from lw64 as a code owner January 8, 2024 23:43
@sonnyp sonnyp merged commit ac687e9 into main Jan 8, 2024
@sonnyp sonnyp deleted the copy-rust-files-on-compile branch January 8, 2024 23:43
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.

Consider using different Cargo.toml for demos

3 participants