-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Policy: New themes going into core? #2157
Comments
Should use same policy as languages: i.e., external repo. |
And what about this idea? I don't think we want to setup individual STYLE repos at the core organization for people, but what about the "discoverability" (which we do seem to care about for languages)? Wouldn't having one big style repo that was pretty much "open to contribution" solve both problems and give us a central place for styles? If we needed to "disclaim" it a little we'd have the name and README that could be used to say "these are unsupported styles from the community, etc"... Thoughts? Creating a GitHub repo on your own that no one is ever going to find doesn't feel much like "contributing" to the project... It feels little bit like we're telling them "frack off, we don't really care you and your style contribution". |
I don't mind to be honest... problem is just having people to maintain things. The intention is to build separate communities and maintainers of this project are also part of the new repo (why we put them in our own organization - so it's not so much "frack off", but more "own your stuff, and we are here to help you... but own your stuff.") |
Well, are you saying we want 1000 CSS repos in the Does themes really require maintenance? If we did add a new class... "burgers"... for tasty new morsels of code... would it be easier to coordinate that with 1000 separate repos or one large style repo? (ie, if someone wanted to pitch in and say add support to all themes - say a burger was really close to a keyword and by default we'd just copy that styling for older themes...) |
I think we're both worried about long-term maintenance, we're just thinking about it differently. Not sure what the right answer is. |
We get very few themes (vs languages), so happy to keep them here. |
True. Ok, why don't you handle a response to #2151 then and then I'll follow your lead on how I respond to themes in the future. |
Closing this with this as the answer if you have no issues, @egor-rogov. |
#2151
What is our policy with new themes? Are there any guidelines on what is required to merge them, or would we prefer all themes be in external repositories (like languages) and no longer accepting new themes to core?
And since themes are much simpler might it be preferable to have a
highlightjs-additional-themes
repository to direct people too and just always accept PRs there? Kind of like a smorgasbord of themes for anyone who wants something not in the default set?The text was updated successfully, but these errors were encountered: