diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 5c9cffc0d..77d556562 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -14,8 +14,9 @@ First of all, thanks for taking the time to contribute! This project can only gr
  • Organizational Guidelines
  • Updating the devicon.json
  • Example of Submitting An Icon
  • +
  • Updating an Icon
  • Maintainer/Reviewer/Teams
  • -
  • The Build Script: how it works and its quirks
  • +
  • Our Workflows: how they work and their issues
  • Discord server
  • Release strategy, conventions, preparation and execution
  • @@ -70,12 +71,14 @@ First of all, thanks for taking the time to contribute! This project can only gr
  • Put the SVGs of each Icon into its own folders in /icons
  • Update the devicon.json to include the new Icon
  • Create a separated pull request (PR) towards the develop branch for each Icon.
  • +
  • Fill out the info as stated in the PR template.
  • Include the name of the Icon in the pull request title in this format: new icon: Icon name (versions)
  • -
  • Optional: Reference the issues regarding the new icon.
  • +
  • Optional: Reference the issues regarding the new icon and label the PR `feature:icon`.
  • Some bots will check your SVGs. If there are any errors, please fix them as instructed.
  • Wait for a maintainer to review your changes. They will run the peek-bot to check your icons.
  • If there are no issues, they will merge it using squash merging. If there are any problems, they will let you know, and give you a chance to fix them.
  • +

    Note: Due to our recent bot upgrades, icon contributors don't have to optimize/minify their SVGs anymore!


    Versions and Naming Conventions

    @@ -326,6 +329,26 @@ As an example, let's assume you have created the SVGs for Redhat and Amazon Web
  • For the rest of the steps, you can follow Overview on Submitting Icon
  • +
    +

    Updating an Icon

    +

    + Sometimes, a company will update their logo or someone spotted an error in the SVG/icon that needs to be fixed. This means the current icon in our repository might need an update. The steps to do this is simple: +

    +
      +
    1. + Create a new commit to fix the SVGs. +
    2. +
    3. + Open a pull request based on the `develop` branch. +
    4. +
    5. + IMPORTANT: name the pull request update icon: icon-name (versions). Basically, follow the Overview on Submitting Icon but replace the new with update in name of request with the above. +
    6. +
    7. + Follow the rest of the steps as laid out in Overview on Submitting Icon. +
    8. +
    +

    Maintainer/Reviewer/Teams

    @@ -410,13 +433,6 @@ As an example, let's assume you have created the SVGs for Redhat and Amazon Web

  • Solution: Ensure the name of the PR follows the convention.
  • -
  • Peek bot fails when an icon is updated - -
  • Icon created by Icomoon contains strange lines that aren't in the SVG
  • diff --git a/README.md b/README.md index 3cdf462c2..4fef76480 100644 --- a/README.md +++ b/README.md @@ -51,6 +51,8 @@
  • Getting started
  • Requesting icon
  • Contributing
  • +
  • Discord server
  • +
  • develop vs master
  • Stale pull requests
  • Go build yourself
  • @@ -159,19 +161,44 @@ Add css rules in your stylesheet

    Contributing

    - We are happy for every contribution. Please have a look at our contribution guidelines - to see how you can contribute to this project. + We are happy for every contribution, whether it's new icons, features, or maintainers. Please have a look at our contribution guidelines to see how you can contribute to this project.

    -

    Stale pull requests

    +

    Discord server

    -After a pull request has been open for over 30 days with no activity or response from the author, it'll be automatically marked as stale. We might fork your changes and merge the changes ourselves. Since GitHub tracks contributions by commits, you will be credited. +We are running a Discord server. You can go here to talk, discuss, and more with the maintainers and other people, too. Here's the invitation: https://discord.gg/hScy8KWACQ. +Note that the Discord server is unofficial, and Devicons is still being maintained via GitHub.

    -

    Discord server

    +

    develop vs master

    -We are running a Discord server. You can go here to talk, discuss, and more with the maintainers and other people, too. Here's the invitation: https://discord.gg/hScy8KWACQ. -Note that the Discord server is unofficial, and Devicons is still being maintained via GitHub. +All official releases shall be in master. Any updates in between (icons, features, etc.) will be kept in develop. +

    +develop contains: + +master contains: + + +

    Stale pull requests

    +

    +After a pull request has been open for over 30 days with no activity or response from the author, it'll be automatically marked as stale. We might fork your changes and merge the changes ourselves. Since GitHub tracks contributions by commits, you will be credited.

    Go build yourself