mirror of
https://github.com/konpa/devicon.git
synced 2025-01-17 13:38:15 +01:00
Update the README and CONTRIBUTING.md (#808)
* Update CONTRIBUTING * Update README * Apply suggestions from code review Co-authored-by: David Leal <halfpacho@gmail.com> Co-authored-by: David Leal <halfpacho@gmail.com>
This commit is contained in:
Thomas Bui
GitHub
parent
08aa325a84
commit
bdad7e4d08
@ -14,8 +14,9 @@ First of all, thanks for taking the time to contribute! This project can only gr
|
||||
<li><a href="#orgGuidelines">Organizational Guidelines</a></li>
|
||||
<li><a href="#updateDevicon">Updating the <code>devicon.json</code></a></li>
|
||||
<li><a href="#example">Example of Submitting An Icon</a></li>
|
||||
<li><a href="#updatingIcons">Updating an Icon</a></li>
|
||||
<li><a href="#teams">Maintainer/Reviewer/Teams</a></li>
|
||||
<li><a href="#buildScript">The Build Script: how it works and its quirks</a></li>
|
||||
<li><a href="#buildScript">Our Workflows: how they work and their issues</a></li>
|
||||
<li><a href="#discordServer">Discord server</a></li>
|
||||
<li><a href="#release">Release strategy, conventions, preparation and execution</a></li>
|
||||
</ul>
|
||||
@ -70,12 +71,14 @@ First of all, thanks for taking the time to contribute! This project can only gr
|
||||
<li>Put the SVGs of each Icon into its own <a href="#orgGuidelines">folders</a> in <code>/icons</code></li>
|
||||
<li><a href="#updateDevicon">Update the <code>devicon.json</code> to include the new Icon</a> </li>
|
||||
<li>Create a separated pull request (PR) towards the <code>develop</code> branch for each Icon.</li>
|
||||
<li>Fill out the info as stated in the PR template.</li>
|
||||
<li>Include the name of the Icon in the pull request title in this format: <code>new icon: <i>Icon name</i> (<i>versions</i>)</code> </li>
|
||||
<li><i>Optional</i>: Reference the issues regarding the new icon. </li>
|
||||
<li><i>Optional</i>: Reference the issues regarding the new icon and label the PR `feature:icon`. </li>
|
||||
<li>Some bots will check your SVGs. If there are any errors, please fix them as instructed.</li>
|
||||
<li>Wait for a maintainer to review your changes. They will run the <a href='#peekBot'><code>peek-bot</code></a> to check your icons.</li>
|
||||
<li>If there are no issues, they will merge it using <a href="https://github.com/devicons/devicon/discussions/470"><b>squash merging</b></a>. If there are any problems, they will let you know, and give you a chance to fix them.</li>
|
||||
</ol>
|
||||
<p><b>Note:</b> Due to our recent bot upgrades, icon contributors don't have to optimize/minify their SVGs anymore!</p>
|
||||
|
||||
<hr>
|
||||
<h2 id='versionNaming'>Versions and Naming Conventions</h2>
|
||||
@ -326,6 +329,26 @@ As an example, let's assume you have created the SVGs for Redhat and Amazon Web
|
||||
<li>For the rest of the steps, you can follow <a href="#overview">Overview on Submitting Icon</a></li>
|
||||
</ol>
|
||||
|
||||
<hr>
|
||||
<h2 id='updatingIcons'>Updating an Icon</h2>
|
||||
<p>
|
||||
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:
|
||||
</p>
|
||||
<ol>
|
||||
<li>
|
||||
Create a new commit to fix the SVGs.
|
||||
</li>
|
||||
<li>
|
||||
Open a pull request based on the `develop` branch.
|
||||
</li>
|
||||
<li>
|
||||
<strong>IMPORTANT</strong>: name the pull request <code>update icon: <i>icon-name</i> (<i>versions</i>)</code>. Basically, follow the <a href="#overview">Overview on Submitting Icon</a> but replace the <code>new</code> with <code>update</code> in name of request with the above.
|
||||
</li>
|
||||
<li>
|
||||
Follow the rest of the steps as laid out in <a href="#overview">Overview on Submitting Icon</a>.
|
||||
</li>
|
||||
</ol>
|
||||
|
||||
<hr>
|
||||
<h2 id='teams'>Maintainer/Reviewer/Teams</h2>
|
||||
<p>
|
||||
@ -410,13 +433,6 @@ As an example, let's assume you have created the SVGs for Redhat and Amazon Web
|
||||
<li>Solution: Ensure the name of the PR follows the convention.</li>
|
||||
</ul>
|
||||
</li>
|
||||
<li><b>Peek bot fails when an icon is updated</b>
|
||||
<ul>
|
||||
<li>See <a href="https://github.com/devicons/devicon/pull/554">this PR</a> for an example.</li>
|
||||
<li>The <code>bot-peek</code> script compares the <code>devicon.json</code> and <code>icomoon.json</code> to limit the icon uploading process. An update in the repo won't change anything in the <code>devicon.json</code> and <code>icomoon.json</code> so the script would report that nothing is found.</li>
|
||||
<li>Solution: Follow the steps laid out <a href="https://github.com/devicons/devicon/pull/554#issuecomment-816860577">here</a></li>
|
||||
</ul>
|
||||
</li>
|
||||
<li><b>Icon created by Icomoon contains strange lines that aren't in the SVG</b>
|
||||
<ul>
|
||||
<li>See <a href="https://github.com/devicons/devicon/pull/532">this PR</a>'s peek result.</li>
|
||||
@ -429,7 +445,7 @@ As an example, let's assume you have created the SVGs for Redhat and Amazon Web
|
||||
<h2 id="discordServer">Discord server</h2>
|
||||
<p>
|
||||
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. If you don't have a GitHub account but want to suggest ideas or new icons, you can do that here in our Discord channel.
|
||||
<b>Note that the Discord server is unofficial, and Devicons is still being maintained via GitHub.</b>
|
||||
<b>The Discord server is unofficial, and Devicons is still being maintained via GitHub.</b>
|
||||
</p>
|
||||
|
||||
<h2 id='release'>Release strategy, conventions, preparation and execution</h2>
|
||||
@ -461,7 +477,7 @@ We are running a Discord server. You can go here to talk, discuss, and more with
|
||||
Take the PRs/commits as a guideline. It's also a good idea to mention and thank all contributions who participated in the release (take description of <code><a href="https://github.com/devicons/devicon/pull/504">#504</a></code> as an example).
|
||||
</li>
|
||||
<li>
|
||||
Rather than doing it manually, you can instead run <code>python ./.github/scripts/get_release_message.py $GITHUB_TOKEN</code> locally. Pass in your GitHub Personal Access Token for <code>$GITHUB_TOKEN</code> and you should see the messages. You can also use the `workflow_dispatch` trigger in the GitHub Actions tab.
|
||||
We now have a script that will do this for us. Check the `build-bot`'s PR message in the last step. There should be a section where it displays the features that have been added to the release. You can copy the markdown there and use it for the release message.
|
||||
</li>
|
||||
</ul>
|
||||
</li>
|
||||
|
||||
43
README.md
43
README.md
@ -51,6 +51,8 @@
|
||||
<li><a href="#getting-started">Getting started</a></li>
|
||||
<li><a href="#request-icon">Requesting icon</a></li>
|
||||
<li><a href="#contribute">Contributing</a></li>
|
||||
<li><a href="#discord-server">Discord server</a></li>
|
||||
<li><a href="#develop-vs-master"><code>develop</code> vs <code>master</code></a></li>
|
||||
<li><a href="#stale-prs">Stale pull requests</a></li>
|
||||
<li><a href="#build-yourself">Go build yourself</a></li>
|
||||
</ol>
|
||||
@ -159,19 +161,44 @@ Add css rules in your stylesheet
|
||||
|
||||
<h2 id="contribute">Contributing</h2>
|
||||
<p>
|
||||
We are happy for every contribution. Please have a look at our <a href="CONTRIBUTING.md">contribution guidelines</a>
|
||||
to see how you can contribute to this project.
|
||||
</p>
|
||||
|
||||
<h2 id="stale-prs">Stale pull requests</h2>
|
||||
<p>
|
||||
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 happy for every contribution, whether it's new icons, features, or maintainers. Please have a look at our <a href="CONTRIBUTING.md">contribution guidelines</a> to see how you can contribute to this project.
|
||||
</p>
|
||||
|
||||
<h2 id="discord-server">Discord server</h2>
|
||||
<p>
|
||||
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.
|
||||
<b>Note that the Discord server is unofficial, and Devicons is still being maintained via GitHub.<b>
|
||||
<b>Note that the Discord server is unofficial, and Devicons is still being maintained via GitHub.</b>
|
||||
</p>
|
||||
|
||||
<h2 id="develop-vs-master"><code>develop</code> vs <code>master</code></h2>
|
||||
<p>
|
||||
All official releases shall be in <code>master</code>. Any updates in between (icons, features, etc.) will be kept in <code>develop</code>.
|
||||
</p>
|
||||
<b><code>develop</code> contains:</b>
|
||||
<ul>
|
||||
<li>
|
||||
Latest SVGs (non-optimized).
|
||||
</li>
|
||||
<li>
|
||||
No icons for the latest SVGs. These will be build at every release.
|
||||
</li>
|
||||
<li>
|
||||
Experimental changes.
|
||||
</li>
|
||||
</ul>
|
||||
<b><code>master</code> contains:</b>
|
||||
<ul>
|
||||
<li>
|
||||
Latest official release, which contains the SVGs and icons.
|
||||
</li>
|
||||
<li>
|
||||
Official tested changes.
|
||||
</li>
|
||||
</ul>
|
||||
|
||||
<h2 id="stale-prs">Stale pull requests</h2>
|
||||
<p>
|
||||
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.
|
||||
</p>
|
||||
|
||||
<h2 id="build-yourself">Go build yourself</h2>
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user