Skip to content
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

Add CONTRIBUTING.md based on our wiki articles #4136

Merged
merged 8 commits into from Feb 18, 2018

Conversation

mrmeszaros
Copy link
Contributor

Created an initial version of CONTRIBUTING.md mentioned in #4132

It contains the contents of the more relevant wiki pages.

Definitely needs some brainstorming, to specify what should go there, and what should be only linked to the wiki.

@bilelmoussaoui
Copy link
Contributor

The file can go under .github btw ;)

@palob palob added the merge label Jan 4, 2018

![inkscape3](https://user-images.githubusercontent.com/7050624/27377856-7afa5020-5676-11e7-93ae-d857b38988ec.png)

## Unthemed icons
Copy link
Contributor

Choose a reason for hiding this comment

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

Quick question: why is the unthemed icons section in CONTRIBUTING.md? I don't see why anyone who is interested in contributing would want this info.

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, you might want to change the README to link to CONTRIBUTING.md and we could drop the wiki page since this pull makes it redundant.

Copy link
Contributor

@Foggalong Foggalong left a comment

Choose a reason for hiding this comment

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

Keep the contribution guidelines for how to submit pull requests and the procedure to follow. In addition to the "Contribution process" add sections for:

  1. New Icon: link to the design guidelines, reminder to update data.json, and inform that user must include Circle & Square and other themes are optional.
  2. Data File: an explanation of the structure and how to add/change it. This includes when adding an icon
  3. Python: we use pycodestyle linting, and basic explanation of @bil-elmoussaoui's validation scripts.

The design guidelines for each theme should stay in the wiki, especially considering they're only going to get longer as new themes are added.

The section on unthemed icons is also irrelivent for the contributing guidelines and should remain a wiki age.

Fixed the broken link to commit message guide.

Added citation blocks for important parts to stand out.
@palob
Copy link
Member

palob commented Jan 6, 2018

Regarding our design guidelines. I'm currently using an Inkscape git build and in the HSL colour notation they changed the ranges for HSL from [0-255,0-255,0-255] to [0-360,0-100,0-100] which afaik is the correct way and how it's done in other graphics applications. This means our baseplate gradient will be 4L instead of 10L as of Inkscape 0.93.

Added the first draft of the requested sections.

The section on Python needs more info from @bil-elmoussaoui.
Extended the optimization options, and turned it into a subsection.

[ci skip]
@palob
Copy link
Member

palob commented Jan 6, 2018

That's gonna be exhaustive, thanks for your efforts!

Structural change to make things read cleaner, removed optimisation settings (probably easier to keep in the wiki), and added external links for more of the definitions and steps.
@Foggalong Foggalong merged commit 2275cf0 into numixproject:master Feb 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

5 participants