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

Add examples to readme #210

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add examples to readme #210

wants to merge 2 commits into from

Conversation

Ben-PH
Copy link

@Ben-PH Ben-PH commented May 17, 2024

I've derived a lot of value from using this crate to bring in constraints on values that can be used, as well as doing compile time arithmetic. I felt these additions to the readme examples would have served me quite well if someone had come before me to make a PR like this, so it's my turn to be that person for those that come after me :)

@paholg
Copy link
Owner

paholg commented May 17, 2024

This is great, thanks! Can you update the full example so that it will compile if you paste it into the rust playground? I think it just needs a few use statement.

Edit: And don't worry about the job failing. I think that's just new to new compiler warnings.

@Ben-PH
Copy link
Author

Ben-PH commented May 26, 2024

I haven't forgotten about your playground edit request. I'll get onto that today.

Maybe also start an examples directory?

@Ben-PH
Copy link
Author

Ben-PH commented May 26, 2024

That should be working now. If you want, I can move the bigger one to an examples directory file, either in this PR or a follow up, as you prefer.

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