Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

added nob.h support to quicksnip under new C category#266

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

Closed
james-beans wants to merge2 commits intoquicksnip-dev:mainfromjames-beans:main
Closed

Conversation

@james-beans
Copy link
Contributor

Description

I have added anob.h sub-category under C called NOB based on configs that I have made to work with thenob.h build system/library.

Type of Change

  • ✨ New snippets
  • 🛠 Improvement to an existing snippet
  • 🐞 Bug fix
  • 📖 Documentation update
  • 🔧 Other (please describe):

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

Note: There was an error when building on my Windows 11 system but it was happening even before I made any changes.

I have added a bit in thepackage.json that should of been there instead of thedev script when starting the project. Just in-case it didn't work on some people's computers. It is calleddev:serve. And yes I know that just runningdev on it's own is good enough but addingdev:serve makes it simpler for contributers if they haven't used Vite before.

Spelling is wrong in the pull request format thing (specifically to do with the type of change bit where it saysnew snippet you need to add an s to the end to make it plural or multiple if a user has made multiple changes).

Screenshots (Optional)

Click to view screenshots

@james-beans
Copy link
ContributorAuthor

As far as I know nob.h doesn't have a logo yet so I put a base there.

Copy link
Collaborator

@Mathys-GasnierMathys-Gasnier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Hey, Thank you for your contribution.

After reviewing your PR we've found that all/some snippets you are adding don't match the scope of Quicksnip.
The snippet you are providing act more like examples. If you have any question or want to contest this decision please send a message in your PR explaining your position.

If no fix is provided within 7-10 days this PR will be closed without further notice

@james-beans
Copy link
ContributorAuthor

@Mathys-Gasnier Most code in quicksnip already are just examples of how someone would implement a function. My code is a config file not a function. If you actually looked at what library it is, you would understand. My code is also an example but so is everything else just in a different format. Thats basically what quicksnip is. A database/website full of examples on how to code things. So why exactly does my code not fit in?

I understand that I repeat things a lot. Please fix my English if it is wrong or mention if I have mentioned the same point more than once. Thank you.

@Mathys-Gasnier
Copy link
Collaborator

We do not provide examples on how to use build systems, at least not for now.
Quicksnip aims to be a library of snippets, which are functions that can be copied in a project without hassle, with a small example of use attached.
Examples like those, aim to show how to use nob, and are more fit for it's documentation.
The scope of quicksnip might expand in the future.
But for now we wont add snippets that are examples of use of build systems/libraries.

FYI: I know what nob is, I followed it's development on tsoding streams.

@james-beans
Copy link
ContributorAuthor

@Mathys-Gasnier

But for now we wont add snippets that are examples of use of build systems/libraries.

Are you sure about the libraries part? There are libraries on quicksnip right now. Examples: React (React itself is a library for JS or TS), FastAPI (A Python library).

I understand now that it doesn't fit at the moment but that should be the thing with stuff like this. It is supposed to expand and is expandable. If you don't start adding stuff like that now it might never be on quicksnip. This is the same with CMake and a lot more stuff that you are currently blocking. You should stop blocking it because it doesn't fit now but might later.

You can already close this if you want. Just respond with a full answer so I can fully understand.

@Mathys-Gasnier
Copy link
Collaborator

The full, simple answer is: We don't accept examples.
Yes we accept librarysnippets but not examples on how to use them.

@james-beans
Copy link
ContributorAuthor

Sorry for being repetitive with things and stuff but that's literally the whole point of quicksnip is to have examples. What you are saying does not make any sense at all to me (a native British person who speaks English) and so I have decided to close this because it goes absolutely no-where. Please if you get the chance to talk to me again (i'm on the discord) please make more sense instead of going in an endless loop of non-understanding and things meaning the same thing.

@technoph1le
Copy link
Collaborator

Hey@james-beans, thank you for your contribution and pointing out a couple of issues. Will fix it soon. xD

I see there's an already discussion with our maintainer,@Mathys-Gasnier. I'm sorry if the explanation didn't come across clearly. Let me explain:

  • Quicksnip focuses on small, reusable function-based snippets that can be copied and pasted into a project with very small changes. More like plug & play.
  • While we do accept snippets involving libraries, we don’t include full usage examples or setup guides.
  • Your snippet is well-structured, but since it’s more of a configuration guide for a build system, it falls outside Quicksnip's current scope.

Yet, as QuickSnip evolves over time, we might explore expanding its scope to include examples like this. If that happens, I’d be happy to revisit this idea.

I appreciate your interest. If you want to chat more, feel free to reach out onDiscord.

No hard feelings, and thanks again for contributing. :)

@james-beans
Copy link
ContributorAuthor

james-beans commentedMar 11, 2025
edited
Loading

@technoph1le


I never have really understood open-source fully and so I haven't contributed at all before to other projects apart from this before. I was the original person to add 'Hello, World!' snippets to most languages that I knew here. Sorry that I didn't understand and am probably wasting both of your time.

Your snippet is well-structured, but since it’s more of a configuration guide for a build system, it falls outside Quicksnip's current scope.

I understand that and am willing to add longer config / snippets to quicksnip using a different way. See below.

I appreciate your interest. If you want to chat more, feel free to reach out onDiscord.

I am willing to talk to you privately on Discord and add a way of adding longer snippets and full config files that would work like mine. My discord isObviouslyimbeans you'll probably find me on there messing around.

No hard feelings, and thanks again for contributing. :)

No thank you for having such a great idea for a website I could contribute to for my first time.

technoph1le reacted with heart emoji

@technoph1le
Copy link
Collaborator

@james-beans, No worries. Thanks to you as well. You already have acontributor tag, so I immediately knew you were a member of our early OG contributors (#17). xD

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@Mathys-GasnierMathys-GasnierMathys-Gasnier requested changes

@majvaxmajvaxAwaiting requested review from majvax

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@james-beans@Mathys-Gasnier@technoph1le

[8]ページ先頭

©2009-2025 Movatter.jp