-
-
Notifications
You must be signed in to change notification settings - Fork 130
Add snippets for Javascript and C #52
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
Conversation
✅ Deploy Preview for quicksnip ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Maybe separate it into different PRs based on the language to make reviewing easier.
Since the changes look good, and the javascript edits are mostly formating issues, i'm going to let this one pass, but please for your futur contributions, separate edits into a PR per language edited.
Okay, i didn't know i had to separate them, also i'm new with contributing so i don't really know how pull requests, forking and all this works. Thanks anyway !
9d33fca
into
quicksnip-dev:main
Okay, i didn't know i had to separate them, also i'm new with contributing so i don't really know how pull requests, forking and all this works. Thanks anyway !
U can create multiple branches for different PRs. I learned it just today as well lol
Description
Added a sleep utility function for javascript
Added the "javascript" tag to some snippets for more consistency between the snippets
Added the C programming Language
Added the Basics category in C with a hello world snippet
Added the Mathematical Functions category in C with a factorial and power snippet
Type of Change
Checklist
Related Issues
Closes #
Additional Context
Screenshots (Optional)
Click to view screenshots