Skip to content
This repository has been archived by the owner on Jul 7, 2020. It is now read-only.

Latest commit

 

History

History
110 lines (86 loc) · 5.16 KB

CONTRIBUTING.md

File metadata and controls

110 lines (86 loc) · 5.16 KB

Contributing to GitHub-Community-Dark

  1. Getting Involved
  2. How To Report style issues
  3. Core Style Guide
  4. Getting Started

Getting Involved

There are a number of ways to get involved with the development of this Dark theme. Even if you've never contributed to an Open Source project before, we're always looking for help identifying missing styles or other issues.

How to Report Style issues

I don't know CSS

If you don't know CSS very well and have found a missing style, please include as much as possible of following information when opening an issue:

  • Screenshot of the problem; include the element(s) in the console if at all possible
    • To select an element, target it with your mouse then right-click and choose "Inspect Element"
    • Please include both the HTML view and the element with the problem in the screenshot (see issue #119 for an example)
  • A URL to the page (if public).

I rock at CSS & GitHub!

  • Follow the style guide below
  • Make any needed changes, then send us a pull request
  • Please include a URL to the page (if public)

Style Guide

  • Use the provided .editorconfig file with your code editor. Don't know what that is? Then check out http://editorconfig.org/.

  • Limit to the K&R (KNF variation style), and 2 SPACE INDENTATION (no tabs, and not more, and not less than 2 spaces).

    • K&R - KNF Variation Example:

      element[attr='value'] {
      ··property: value;
      }
    • Not Allman

      element[property='value']
      {
      ··property: value;
      }
    • Strict space between the selector and the {:

      /* good */
      element[attr='value'] { }
      
      /* bad */
      element[attr='value']{ }
    • 2 Space indentation

      /* good */
      ··property: value;
      
      /* bad */
      ····property: value;
      ----property: value;
      ·property: value;
  • Try to wrap lines at around 80 characters.

  • Try to limit the style size:

    • Don't add any image URI's to the CSS; instead add the image into the /images directory; then point to using the following URL: http://stylishthemes.github.io/GitHub-Community-Dark/images/{my-image.png}.
    • If possible, reduce any added selectors. Remember that Stylus requires an !important flag to override default styling, so a selector starting from the body isn't always necessary.
    • Don't add any inline comments. If you want to make a comment, add it as a note in the commit.
    • If your CSS definition already exists within the style, do not add it again! Add your selector to the existing definition.
  • Insert any new CSS selectors in the available slot immediately before the style definition, or on a new line as needed.

  • If you want to add a new userstyle or usercss variable, please open an issue and discuss it with us first.

  • Don't include version bumps with your contribution, all releases are handled internally.

  • If your PR fixes an open issue or replaces another PR, include fixes/closes #issue-nr in your commit message title. Read more on this.

Getting Started

  • Download, fork or clone this repository.
  • Use node.js to run npm install.
  • Make any changes to the github-community-dark.user.css file and save.

Build & test

  • Create & change into a new branch of your local GitHub-Community-Dark repository.
  • Open the style in the Stylus editor, and make sure to have "live preview" checked for testing.
  • Once you are satisfied with the changes, select all the CSS (Ctrl + a), copy (Ctrl + c) then paste (Ctrl + v) it into your editor.
  • Run npm test to test the CSS changes.
  • Now you can add and commit the changes of the github-community-dark.user.css file to your fork's branch.
  • If you haven't already contributed, then run npm run authors to add your name to our list of contributors 😄
  • Push the changes to your branch, then submit a pull request.
  • And thanks again for contributing!

Development Scripts

  • npm run authors: Runs a batch file to rebuild the AUTHORS file. Update the .mailmap file for any duplicate entries.
  • npm run clean: Runs the perfectionist script & cleans up after it.
  • npm run eslint: Lint the JavaScript code in the tools directory.
  • npm run lint: Run eslint & stylelint scripts.
  • npm run major: Creates a semantic major release.
  • npm run minor: Creates a semantic minor release.
  • npm run patch: Creates a semantic patch release.
  • npm run perfectionist: Runs perfectionist only. CSS is not cleaned!
  • npm run stylelint: Run stylelint on the CSS file.
  • npm run test: Same as npm run lint.
  • npm run update: Update development dependencies.
  • npm run usercss: Update usercss variables using usercss template; variable data obtained from defaults.json.