-
-
Notifications
You must be signed in to change notification settings - Fork 135
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 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.
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!
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.
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. |
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. |
One idea would be to name the category "Beginners" just showing the minimal snippet of code needed to print an |
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? |
Did you push ? I can't see the change on your repo so you probably forgot |
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? |
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. |
Ok ill change it. That does kind of sound like a good name now after thinking about it. |
Done it! I might add more later but I will add to another pr instead. |
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.