Skip to content

Commit 6b81f3c

Browse files
committed
docs: update contributing + readme, add changelog
1 parent 11ad21c commit 6b81f3c

File tree

4 files changed

+71
-63
lines changed

4 files changed

+71
-63
lines changed

CHANGELOG.md

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
# Changelog
2+
3+
All versions upwards from `2.0.0` project will be documented in this file. For all older version, please check the [Releases](https://github.com/marcopixel/r6operators/releases/) tab.
4+
5+
## 2.0.0 (2021-09-08)
6+
7+
### ✨ Features
8+
9+
- 📝 Rewrote library code for better typings, smaller filesize and future features ([6c84917](https://github.com/marcopixel/r6operators/commit/6c8491729761d3929a55597b655f41d091cce0ec))
10+
- 🎨 Reworked existing operator icons to the new style ([bfac854](https://github.com/marcopixel/r6operators/commit/bfac85442b01429553996428659bae1eee87ffb6), [80dbc5b](https://github.com/marcopixel/r6operators/commit/80dbc5b577449bf39dc50175115fb86c7aa410ea))
11+
- 🌟 Added Thorn opperator icon & metadata ([11ad21c](https://github.com/marcopixel/r6operators/commit/11ad21c2ae109c867203bc5782cf6bb3804cfb8f))
12+
13+
### 🐛 Bug Fixes
14+
15+
- 🛠 Corrected incorrect metadata on Doc & Thatcher ([bb7e7a5](https://github.com/marcopixel/r6operators/commit/bb7e7a5d185026dfddb24f58da01295f123fa52a))
16+
17+
### 🚨 Breaking Changes
18+
19+
- **BREAKING**: Changed `[op].sex` to `[op].gender` ([b46431e](https://github.com/marcopixel/r6operators/commit/b46431e2e2181083c120948161fe91c77e4fc4cf))
20+
- **BREAKING**: Renamed `[op].ratings.armor` to `[op].ratings.health` ([4311beb](https://github.com/marcopixel/r6operators/commit/4311beb57cc80ae2562631c83e6d6bbf2bad2b0a))
21+
- **BREAKING**: PNG files are no longer supplied with the library ([See #1 for more information](https://github.com/marcopixel/r6operators/issues/1#issuecomment-861881904))
22+
- **BREAKING**: Compiled library & icons path moved from `/lib` to `/dist` ([7756724](https://github.com/marcopixel/r6operators/commit/7756724e0f2d56cf7cc8521da8727c0804af06da))
23+
24+
Thanks to [@danielwerg](https://github.com/danielwerg) for his help on this release 👍

CONTRIBUTING.md

Lines changed: 44 additions & 60 deletions
Original file line numberDiff line numberDiff line change
@@ -4,92 +4,76 @@ Thank you for your interest to contribute to this project. You're awesome! :+1:
44

55
Below here are some guidelines for contributing to this project, please read them before creating a pull request/issue on Github.
66

7-
Feel free to improve these guidelines with a pull request :)
7+
Feel free to improve these guidelines with a pull request! 😄
88

9-
## Creating the icon
9+
## Getting started
1010

11-
> **Note:** We recommend using Adobe Illustrator for this project. If you do not own Illustrator please make the icon boundaries (white border of the icon) exactly 350 x 350 pt.
11+
**What does "contributing" mean?**
1212

13-
If you want to create an icon, please use the [template.ai](https://github.com/marcopixel/r6operators/blob/master/src/template.ai) file which is located in the repository root folder.
13+
Creating an issue is the simplest form of contributing to a project. But there are many ways to contribute, including the following:
1414

15-
The template includes the necessary guides for proper sizing & file layout, which we recommend for you to use since it makes every ones work so much easier :)
15+
- Updating or correcting documentation
16+
- Feature requests
17+
- Bug reports
1618

17-
**Here's an screenshot of the template loaded into Illustrator CC:**
19+
If you'd like to learn more about contributing in general, the [Guide to Idiomatic Contributing](https://github.com/jonschlinkert/idiomatic-contributing) has a lot of useful information.
1820

19-
> ![Illustrator example](https://i.imgur.com/xpZd4Vs.png)
21+
## Issues
2022

21-
Also follow these guidelines:
23+
Please only create issues for bug reports or feature requests. Issues discussing any other topics may be closed by the project's maintainers without further explanation.
2224

23-
- Keep icon boundaries (white border) inside the guides
24-
- Keep all paths/shapes inside the artboard
25-
- Rename the artboard & layer to the `OperatorName`
25+
Do not create issues about bumping dependencies unless a bug has been identified and you can demonstrate that it effects this library.
2626

27-
**Example screenshot of a finished icon:**
27+
**Help us to help you**
2828

29-
> ![Finished example screenshot](https://i.imgur.com/ZKbBPSe.png)
29+
Remember that we’re here to help, but not to make guesses about what you need help with:
3030

31-
After you're finished with your icon, save the icon under the `icons` folder (filename in lower-case) in the root directory and continue with the steps down below.
31+
- Whatever bug or issue you're experiencing, assume that it will not be as obvious to the maintainers as it is to you.
32+
- Spell it out completely. Keep in mind that maintainers need to think about _all potential use cases_ of a library. It's important that you explain how you're using a library so that maintainers can make that connection and solve the issue.
3233

33-
> Info: If you're using Illustrator, please save your file with the following settings:
34-
>
35-
> - SVG Profiles: **SVG 1.1**
36-
> - Type: **SVG**
37-
> - Subsetting: **None (Use System Fonts)**
38-
> - Image Location: **Embed**
39-
> - Preserve Illustrator Editing Capabilities **ON**
40-
> - CSS Properties: **Style Attributes**
41-
> - Decimal Places: **3**
42-
> - Encoding: **Unicode (UTF-8)**
43-
> - Output fewer <tspan> elements **ON**
44-
> - Use <textPath> element for Text on Path **ON**
45-
> - Responsive **ON**
46-
> - Include Slicing Data **OFF**
47-
> - Include XMP **OFF**
34+
_It can't be understated how frustrating and draining it can be to maintainers to have to ask clarifying questions on the most basic things, before it's even possible to start debugging. Please try to make the best use of everyone's time involved, including yourself, by providing this information up front._
4835

49-
## Adding the icon to the package
36+
### Before creating an issue
5037

51-
> **Note:** This guide requires a little knowledge about Node.js, PATH and Git, if you feel unsure about this skip this step and just create a pull request with the icon only.
38+
Please try to determine if the issue is caused by an underlying library, and if so, create the issue there. Sometimes this is difficult to know. We only ask that you attempt to give a reasonable attempt to find out. Oftentimes the readme will have advice about where to go to create issues.
5239

53-
#### Requirements:
40+
Try to follow these guidelines:
5441

55-
- [Node.js](https://nodejs.org/en/)
56-
- [Yarn](https://yarnpkg.com/)
57-
- [Inkscape](https://inkscape.org/) (installed on [PATH](https://superuser.com/questions/284342/what-are-path-and-other-environment-variables-and-how-can-i-set-or-use-them), you can use [chocolatey](https://chocolatey.org/packages/InkScape) to automatically install & add it on Windows)
42+
- **Avoid creating issues for implementation help** - It's much better for discoverability, SEO, and semantics - to keep the issue tracker focused on bugs and feature requests - to ask implementation-related questions please go to [stackoverflow.com](https://stackoverflow.com/)
43+
- **Investigate the issue** - Search for exising issues (open or closed) that address the issue, and might have even resolved it already.
44+
- **Check the readme** - oftentimes you will find notes about creating issues, and where to go depending on the type of issue.
45+
- Create the issue in the appropriate repository.
5846

59-
To setup & install all dependencies for the project, install [Node.js](https://nodejs.org/en/) & [Yarn](https://yarnpkg.com/) and run the following commands in the root directory:
47+
### Creating an issue
6048

61-
```shell
62-
yarn install
63-
```
49+
Please be as descriptive as possible when creating an issue. Give us the information we need to successfully answer your question or address your issue by answering the following in your issue:
6450

65-
Now open your favorite code editor (we recommend [Visual Studio Code](https://code.visualstudio.com/)) and go to the [`operators.json`](https://github.com/marcopixel/r6operators/blob/master/src/operators.json) file located under the `src` folder.
51+
- **description**: (required) What is the bug you're experiencing? How are you using this library/app?
52+
- **version**: (required) please note the version you are using
53+
- **error messages**: (required) please paste any error messages into the issue, or a [gist](https://gist.github.com/)
6654

67-
There you will find a huge JSON object containing all metadata for all the operators sorted alphabetically (except the recruits, those are at the bottom). From there you just have to create a new object with the same name as your `icon.svg` and fill out the metadata.
55+
### Closing issues
6856

69-
> ![Screenshot of the JSON file](https://i.imgur.com/HBljxs1.png)
57+
The original poster or the maintainers may close an issue at any time. Typically, but not exclusively, issues are closed when:
7058

71-
If you did everything correctly, you just have to enter `yarn build` into the command line and it should start the build process automatically. This will clean up any old generated files, generate the icons + javascript files and test them for any errors.
59+
- The issue is resolved
60+
- The project's maintainers have determined the issue is out of scope
61+
- An issue is clearly a duplicate of another issue, in which case the duplicate issue will be linked.
62+
- A discussion has clearly run its course
7263

73-
Your icons should show up under `lib/icons` in their respective folders and if no errors show up you can go and commit the changes to your pull request.
64+
## Next steps
7465

75-
## Submitting an icon
66+
**Tips for creating idiomatic issues**
7667

77-
The best way to contribute an icon is by creating a pull request. They allow for better collaboration and also allow for proper reviews.
68+
Spending just a little extra time to review best practices and brush up on your contributing skills will, at minimum, make your issue easier to read, easier to resolve, and more likely to be found by others who have the same or similar issue in the future. At best, it will open up doors and potential career opportunities by helping you be at your best.
7869

79-
**Working on your first Pull Request?** You can learn how from this _free_ series [How to Contribute to an Open Source Project on GitHub](https://egghead.io/series/how-to-contribute-to-an-open-source-project-on-github)
70+
The following resources were hand-picked to help you be the most effective contributor you can be:
8071

81-
## Running tests
72+
- The [Guide to Idiomatic Contributing](https://github.com/jonschlinkert/idiomatic-contributing) is a great place for newcomers to start, but there is also information for experienced contributors there.
73+
- Take some time to learn basic markdown. We can't stress this enough. Don't start pasting code into GitHub issues before you've taken a moment to review this [markdown cheatsheet](https://gist.github.com/jonschlinkert/5854601)
74+
- And if you want to really go above and beyond, read [mastering markdown](https://guides.github.com/features/mastering-markdown/).
8275

83-
To run the tests from the command line, simply run the following command:
76+
At the very least, please try to:
8477

85-
```shell
86-
yarn test
87-
```
88-
89-
This will start all tests automatically and will tell you exactly where something went wrong.
90-
91-
## Icon requests
92-
93-
Before creating an icon request, please go to issues and search if someone requested the icon already. If there is an open request, please add a ":+1:" to the main post.
94-
95-
If the icon has not already been requested, [create an issue](https://github.com/marcopixel/r6operators/issues/new?title=Icon%20Request:) with a title of `Icon request: <name>` and add as much information as possible (if possible also add images of the icon, so we can start as soon as possible)
78+
- Use backticks to wrap code. This ensures that it retains its formatting and isn't modified when it's rendered by GitHub, and makes the code more readable to others
79+
- When applicable, use syntax highlighting by adding the correct language name after the first "code fence"

LICENSE

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
MIT License
22

3-
Copyright (c) 2020 Marco Vockner
3+
Copyright (c) 2021 Marco Vockner
44

55
Permission is hereby granted, free of charge, to any person obtaining a copy
66
of this software and associated documentation files (the "Software"), to deal
@@ -18,4 +18,4 @@ FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
1818
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
1919
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
2020
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
21-
SOFTWARE.
21+
SOFTWARE.

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ Caught a mistake or want to contribute to the documentation? [Edit this page on
185185
- [@colebemis](https://github.com/colebemis) for his work on [feather](https://github.com/feathericons/feather), which gave me an awesome reference for this project.
186186
- [@dtSniper](https://twitter.com/sniperdt) for creating the IQ, Thatcher, Fuze, Glaz, Bandit, Kapkan, Tachanka, Pulse, Sledge and Doc icons.
187187
- [@joeyfjj](https://twitter.com/joeyfjj) for creating the Goyo, Mute, Smoke, Jäger and Blitz icons.
188-
- [@LaxisB](https://github.com/LaxisB/) & [@NaughtyMuppet](https://github.com/NaughtyMuppet) for general help on this project. <3
188+
- [@LaxisB](https://github.com/LaxisB/), [@NaughtyMuppet](https://github.com/NaughtyMuppet) & [@danielwerg](https://github.com/danielwerg) for general help on this project. <3
189189

190190
## License
191191

0 commit comments

Comments
 (0)