Skip to content

Conversation

0xHouss
Copy link
Contributor

@0xHouss 0xHouss commented Dec 31, 2024

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

  • ✨ New snippet
  • 🛠 Improvement to an existing snippet
  • 🐞 Bug fix
  • 📖 Documentation update
  • 🔧 Other (please describe): Adds the C programming language.

Checklist

  • I have tested my code and verified it works as expected.
  • My code follows the style and contribution guidelines of this project.
  • Comments are added where necessary for clarity.
  • Documentation has been updated (if applicable).
  • There are no new warnings or errors from my changes.

Related Issues

Closes #

Additional Context

Screenshots (Optional)

Click to view screenshots

Copy link

netlify bot commented Dec 31, 2024

Deploy Preview for quicksnip ready!

Name Link
🔨 Latest commit a00df69
🔍 Latest deploy log https://app.netlify.com/sites/quicksnip/deploys/6773d462b88d11000863c649
😎 Deploy Preview https://deploy-preview-52--quicksnip.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@Axorax Axorax left a 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.

image

@Mathys-Gasnier Mathys-Gasnier added Snippets update needed Code needs to be updated. labels Dec 31, 2024
@Mathys-Gasnier
Copy link
Collaborator

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.

@0xHouss
Copy link
Contributor Author

0xHouss commented Dec 31, 2024

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 !

@Mathys-Gasnier Mathys-Gasnier merged commit 9d33fca into quicksnip-dev:main Dec 31, 2024
4 checks passed
@Axorax
Copy link
Contributor

Axorax commented Dec 31, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Snippets update needed Code needs to be updated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants