- Notifications
You must be signed in to change notification settings - Fork128
Added Hello, World! example to Python, Rust, Javascript and C++#17
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
netlifybot commentedDec 29, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
✅ Deploy Preview forquicksnip ready!
To edit notification comments on pull requests, go to yourNetlify site configuration. |
wneel left a comment
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.
I would maybe change the category name to something like "basics" or just "Hello World" but this is a great idea!
- Idk if this is wanted or not but you just put a line break in the .cpp!
james-beans commentedDec 29, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Yeah if you were talking about under the import of the cpp implementation then yes there is supposed to be an empty line.
I think that it being called "Terminal Output" is just fine on all of them apart from the JavaScript implementation because of it being used in a lot of other things other than just CLI terminal tools. Because of this, I might change the JavaScript snippet category name to just "Output" if that's ok. |
changed the category name "terminal output" to just "output" in the category name of JavaScript.
technoph1le commentedDec 30, 2024
Hey, there. Thanks for the contribution. Regarding your idea, I like it. I think we can change the category to "Basics" and include basic and handy code snippets in programming languages that doesn't belong to any category. Also, designed for those who are new to that programming language. It would also keep it consistent across other languages as well. I'm open to listening more ideas as well. |
james-beans commentedDec 30, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I think that having it just called 'Basics' is a bit more general vs what it actually does which is terminal output. Renaming it could make it a bit harder to organize snippets without another type of category that goes under another category (sub-categories). Adding a search feature for tags would be nice to have especially for snippets if we are going to make sub-categories. |
Mathys-Gasnier commentedDec 30, 2024
One idea would be to name the category "Beginners" just showing the minimal snippet of code needed to print an |
Uh oh!
There was an error while loading.Please reload this page.
james-beans commentedDec 30, 2024
I added the fix for it to my forked version of this repo and it used to automatically add the changes but it hasn't seemed to update the latest commit to this? Can someone add it for me? |
Mathys-Gasnier commentedDec 30, 2024
Did you push ? I can't see the change on your repo so you probably forgot |
james-beans commentedDec 30, 2024
I did commit it. It just didn't update for like 2 hours straight on this pr. It has now. Can you check if this is finally done so someone can push it? |
Mathys-Gasnier commentedDec 30, 2024
Before anything I would like you to changes the categories of the snippets you added, I think we came to a conclusion that a category named "Basics" fits better those snippets. |
james-beans commentedDec 30, 2024
Ok ill change it. That does kind of sound like a good name now after thinking about it. |
james-beans commentedDec 30, 2024
Done it! I might add more later but I will add to another pr instead. |
0615529 intoquicksnip-dev:mainUh oh!
There was an error while loading.Please reload this page.
I have added a Hello, World! example to all of the programming languages that I know have a printing function. This also might be a bit to beginners like for what you have already put on the languages.
Please tell me if I have messed something up. I think this should all be correct and working.