-
-
Notifications
You must be signed in to change notification settings - Fork 139
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
feat: enhance Rust snippets #75
base: main
Are you sure you want to change the base?
Conversation
β Deploy Preview for quicksnip ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Hey, we recently updated how snippets are stored, please update your PR to match the changes, you will be able to see how to add snippets in |
Hi @Mathys-Gasnier, Iβve updated the snippets to match the new format and resolved the merge conflicts. Can you take a look and let me know if everythingβs good or if thereβs anything else I should fix? Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, thanks for updating your snippets, but looking at your PR, it still seems to modify /public/consolidated/all_snippets.json
which is a file that doesn't exists, the same goes for /public/data/rust.json
Could you fix the conflict to rust.json please ? |
97c58d2
to
1c1c0d1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
Description
Type of Change
Checklist
Related Issues
Closes #
Additional Context
The changes introduce a variety of new Rust snippets.
Screenshots (Optional)
Click to view screenshots