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

[PLATFORM-866]: Update examples and documentation in Veil for all the new changes #65

Conversation

WilliamVenner
Copy link
Contributor

@WilliamVenner WilliamVenner requested a review from a team as a code owner December 15, 2022 15:02
.drone.yml Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
examples/enums.rs Outdated Show resolved Hide resolved
veil-macros/src/lib.rs Outdated Show resolved Hide resolved
@WilliamVenner WilliamVenner force-pushed the PLATFORM-866/user-story/update-examples-and-documentation-in-veil-for-all-the-new-changes branch from b85a3de to de9b57e Compare December 22, 2022 11:19
@WilliamVenner
Copy link
Contributor Author

Looks like the Dronefile was formatted by my IDE. I didn't even know you could format a YAML file incorrectly. Oh well

@MaeIsBad
Copy link
Member

I think you can completely remove the signature section of the drone file by the way

@WilliamVenner
Copy link
Contributor Author

Yeah it needs to be resigned now. I think we should do that in this PR. Are we able to self-sign them now, or do we need to ask someone?

@MaeIsBad
Copy link
Member

Oh right, I thought drone just let the CI through on veil. I think only @cpiemontese should be able to sign it, but for whatever reason drone also lets me do it, so 🤷‍♀️

.drone.yml Outdated Show resolved Hide resolved
.drone.yml Outdated Show resolved Hide resolved
MaeIsBad
MaeIsBad previously approved these changes Dec 22, 2022
Copy link
Member

@MaeIsBad MaeIsBad left a comment

Choose a reason for hiding this comment

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

one small thing, other than that 👍

examples/manual_builder.rs Outdated Show resolved Hide resolved
Co-authored-by: MaeIsBad <26093674+MaeIsBad@users.noreply.github.com>
@WilliamVenner
Copy link
Contributor Author

Ok then... not sure why it dismissed the review when it was a committed suggestion.

@WilliamVenner WilliamVenner merged commit 390a4c9 into master Dec 22, 2022
@WilliamVenner WilliamVenner deleted the PLATFORM-866/user-story/update-examples-and-documentation-in-veil-for-all-the-new-changes branch December 22, 2022 11:46
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