mirror of
https://github.com/konpa/devicon.git
synced 2025-03-13 17:29:42 +01:00
Add alphabetical ordering of devicon.json
to PR checklist (#1332)
* Add note to maintainers about checking devicon.json related #1327 * implement suggest changes from PR reviews * fix updating devicon.json wiki link
This commit is contained in:
parent
eb35d73a01
commit
300685abbf
2
.github/PULL_REQUEST_TEMPLATE/new_icon.md
vendored
2
.github/PULL_REQUEST_TEMPLATE/new_icon.md
vendored
@ -6,7 +6,7 @@
|
||||
- [ ] PR's base is the `develop` branch.
|
||||
- [ ] Your icons are inside a folder as seen [here](https://github.com/devicons/devicon/wiki/Organizing-SVGs)
|
||||
- [ ] SVG matches the standards laid out [here](https://github.com/devicons/devicon/wiki/SVG-Standards)
|
||||
- [ ] A new object is added in the `devicon.json` file as seen [here](https://github.com/devicons/devicon/wiki/Updating-the-%60devicon.json%60)
|
||||
- [ ] A new object is added in the `devicon.json` file at the correct alphabetic position as seen [here](https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60)
|
||||
|
||||
<!-- Refer to the [contributing](https://github.com/devicons/devicon/blob/develop/CONTRIBUTING.md#contributing-to-devicon) guidelines for more details. -->
|
||||
|
||||
|
3
.github/workflows/post_peek_screenshot.yml
vendored
3
.github/workflows/post_peek_screenshot.yml
vendored
@ -109,6 +109,7 @@ jobs:
|
||||
1. The number of Glyphs matches the number of SVGs that were selected.
|
||||
2. The icons (second group of pictures) look the same as the SVGs (first group of pictures).
|
||||
3. The icons are of high quality (legible, matches the official logo, etc.)
|
||||
4. A new object is added in the `devicon.json` file at the correct alphabetic position as seen [here](https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60)
|
||||
|
||||
In case of font issues, it might be caused by Icomoon not accepting strokes in the SVGs. Check this [doc](https://icomoon.io/#faq/importing) for more details and fix the issues as instructed by Icomoon and update this PR once you are done.
|
||||
|
||||
@ -151,7 +152,7 @@ jobs:
|
||||
Make sure that:
|
||||
|
||||
- Your SVGs are named and added correctly to the /icons folder as seen [here](https://github.com/devicons/devicon/wiki/Organizing-SVGs).
|
||||
- Your icon information has been added to the `devicon.json` as seen [here](https://github.com/devicons/devicon/wiki/Updating-the-%60devicon.json%60)
|
||||
- Your icon information has been added to the `devicon.json` as seen [here](https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60)
|
||||
- Your PR title follows the format seen [here](https://github.com/devicons/devicon/wiki/Overview-on-Submitting-Icons)
|
||||
|
||||
I will retry once everything is fixed. If I still fail or there are other error, the maintainers will investigate.
|
||||
|
Loading…
x
Reference in New Issue
Block a user