Skip to content

Commit 4c7f496

Browse files
authored
Merge branch 'gh-pages' into 312-add-donation-button
2 parents 3956982 + 75e3c90 commit 4c7f496

File tree

10 files changed

+314
-209
lines changed

10 files changed

+314
-209
lines changed

CONTRIBUTING.md

Lines changed: 91 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -4,35 +4,83 @@ lang: en
44
title: Contributing
55
---
66

7-
# Contributing to Black Python Devs
7+
# Contributing at Black Python Devs
88

9-
First off, thank you for considering contributing to Black Python Devs website. It's people like you that make Black Python Devs such a great community.
9+
First off, thank you for considering contributing to Black Python Devs website. It's people like you that make Black Python Devs such a great community. Navigate through the following to understand more about contributing here.
10+
11+
- [Before You Get Started](#before-you-get-started)
12+
- [How to Contribute](#how-to-contribute)
13+
- [Accessibility](#accessibility)
14+
- [Translations](#translations)
15+
16+
# Before You Get Started
1017

1118
## Code of Conduct
1219

13-
Help us keep this project open and inclusive. Please read and follow our [Code of Conduct](https://github.com/BlackPythonDevs/.github/blob/main/CODE_OF_CONDUCT.md).
20+
Black Python Devs follows the following [Code of Conduct](https://github.com/BlackPythonDevs/.github/blob/main/CODE_OF_CONDUCT.md) . The comfort and safety of Black Python Devs community members are our priority. Please do well to adhere to the Code of Conduct to participate in the Black Python Devs community.
1421

15-
## How Can I Contribute?
22+
## Issues & Pull Requests
1623

17-
### 1. Create an Issue (and wait to be assigned)
24+
### Creating an issue
1825

19-
- Click on the Issues tab in the repository.
26+
Before **creating** an issue i.e for `features`/`bugs`/`improvements` please follow these steps:
2027

21-
![Issues tab](/assets/images/issues_tab.png)
28+
1. Search existing issues before creating a new issue (look to see if the issue has already been created).
29+
1. If it doesn't exist create a new issue giving as much context as possible (please take note and select the correct issue type, for example `bug`, `documentation` or `feature`.
30+
1. If you wish to work on the issue add this to the issue description.
2231

23-
- Click on the green button with the label **"New issue"** and you should see the image below:
32+
### Working on an issue
2433

25-
![New issue button](/assets/images/new_issue.png)
34+
Before working on an existing issue please follow these steps:
2635

27-
- Type in the issue title in the title textbox and type in a clear explanation of the issue in the textarea. You can add images, code snippets, etc to explain your issue.
36+
1. Comment asking for the issue to be assigned to you.
37+
1. To best position yourself for issues assignment, we recommend that you:
38+
1. Confirm that you have read the CONTRIBUTING.md.
39+
1. Have a functional development environment (have built and are able to run the project).
40+
1. Convey your intended approach to solving the issue.
41+
1. Put each of these items in writing in one or more comments.
42+
1. After the issue is assigned to you, you can start working on it.
43+
1. In general, **only** start working on this issue (and open a Pull Request) when it has been assigned to you. Doing so will prevent confusion, duplicate work (some of which may go unaccepted given its duplicity), incidental stepping on toes, and the headache involved for maintainers and contributors alike as issue assignments collide and heads bump together.
44+
1. Reference the issue in your Pull Request (for example `This PR fixes #123`), so that the corresponding issue is automatically closed upon merge of your Pull Request.
45+
46+
> Notes:
47+
>
48+
> - Check the `Assignees` box at the top of the page to see if the issue has been assigned to someone else before requesting this be assigned to you. If the issue has a current Assignee, but appears to be inactive, politely inquire with the current Assignee as to whether they are still working on a solution and/or if you might collaborate with them.
49+
> - Only request to be assigned an issue if you know how to work on it.
50+
> - If an issue is unclear, ask questions to get more clarity before asking to have the issue assigned to you; avoid asking "what do I do next? how do I fix this?" (see the item above this line)
51+
> - An issue can be assigned to multiple people, if you all agree to collaborate on the issue (the Pull Request can contain commits from different collaborators)
52+
> - Any issues that has no activity after 2 weeks will be unassigned and re-assigned to someone else.
53+
54+
## Reviewing Pull Requests
55+
56+
We welcome everyone to review Pull Requests. It is a great way to learn, network, and support each other.
57+
58+
### DOs
59+
60+
- Use inline comments to explain your suggestions
61+
- Use inline suggestions to propose changes
62+
- Exercise patience and empathy while offering critiques of the works of others.
63+
64+
### DON'Ts
65+
66+
- Do not repeat feedback, this creates more noise than value (check the existing conversation), use GitHub reactions if you agree/disagree with a comment
67+
- Do not blindly approve Pull Requests to improve your GitHub contributors graph
68+
69+
## Discord Community
70+
71+
Join the [Discord](https://discord.gg/XUc3tFqCT3) to discuss suggested new features, possible bugs, enhancement in user experience, and any other aspects of the site. The comment section of each issue is our preferred method of communication as it retains conversations history for future contributors wanting to gain insights/updates on the topic in question, you can, however, inquire in the #community-discussion channel in the Black Python Devs Discord workspace.
2872

29-
- Once you are done, you can preview your issue by selecting the Preview tab. If you are satisfied with the issue presentation, you can click on the green "Submit new issue" button to create ypur issue.
73+
# Contributing to Black Python Devs Projects
3074

31-
- Now you wait for your issue to be assigned to you by a maintainer. You should get an email notification of the issue being assigned to you or you can view the issue assignment in the issue page. It should look similar to this:
75+
Please follow these steps and note these guidelines to begin contributing:
3276

33-
![Issue assignment](/assets/images/issue_assignment.png)
77+
1. First step is to set up the local development environment.
78+
1. Bug fixes are always welcome. Start by reviewing the [list of bugs](https://github.com/BlackPythonDevs/blackpythondevs.github.io/issues).
79+
1. A good way to easily start contributing is to pick and work on a [good first issue](https://github.com/BlackPythonDevs/blackpythondevs.github.io/labels/good%20first%20issue). We try to make these issues as clear as possible and provide basic info on how the code should be changed, and if something is unclear feel free to ask for more information on the issue.
3480

35-
### 2. Fork the Code
81+
# How to Contribute
82+
83+
## Fork the repository
3684

3785
- To fork the repository so you have a copy of the codebase, you will click on the **"Fork"** button from the repository main page
3886

@@ -42,7 +90,25 @@ Help us keep this project open and inclusive. Please read and follow our [Code o
4290

4391
![Create new fork page](/assets/images/create_new_fork_page.png)
4492

45-
### 3. Make the changes
93+
## Createing an issue
94+
95+
- Click on the issues tab in the repository.
96+
97+
![issues tab](/assets/images/issues_tab.png)
98+
99+
- Click on the green button with the label **"New issue"** and you should see the image below:
100+
101+
![New issue button](/assets/images/new_issue.png)
102+
103+
- Type in the issue title in the title textbox and type in a clear explanation of the issue in the textarea. You can add images, code snippets, etc to explain your issue.
104+
105+
- Once you are done, you can preview your issue by selecting the Preview tab. If you are satisfied with the issue presentation, you can click on the green "Submit new issue" button to create your issue.
106+
107+
- And that's it! If you cose to comment requesting the issue to be assigned to you, wait for a maintainer to do so. You should get an email notification of the issue being assigned to you or you can view the issue assignment in the issue page. It should look similar to this:
108+
109+
![issue assignment](/assets/images/issue_assignment.png)
110+
111+
## Working on an issue
46112

47113
- The easiest way to make changes and and test them is by using [![GitHub Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/BlackPythonDevs/blackpythondevs.github.io)
48114

@@ -52,7 +118,7 @@ Help us keep this project open and inclusive. Please read and follow our [Code o
52118

53119
- The web version of VSCode should open in a new tab in your browser:
54120

55-
![BlackPythonDevs Codespace](/assets/images/blackpythondevs_codespace.png)
121+
![Black Python Devs Codespace](/assets/images/BlackPythonDevs_codespace.png)
56122

57123
- Now that you have the code editor set up, you need to install the dependencies. To do this, you have to open the code editor's terminal and run the command `bundle install`.
58124

@@ -78,41 +144,39 @@ Help us keep this project open and inclusive. Please read and follow our [Code o
78144

79145
- Test your changes (create new tests as needed)
80146

81-
- Submit a Pull Request
82-
83-
🥳
147+
- Once you’ve committed and pushed all of your changes to GitHub, go to the page for your fork on GitHub, select your development branch, and click the pull request button. Please ensure that you compare your feature branch to the desired branch of the repo you are supposed to make a PR to. If you need to make any adjustments to your pull request, just push the updates to GitHub. Your pull request will automatically track the changes in your development branch and update it. 🥳
84148

85-
## Accessibility
149+
# Accessibility
86150

87151
Accessibility (A11y) using the **FastPass** tests for A11y and the [Accessibility Insights for Web
88152
][1] browser extension.
89153

90-
### 1: Reproduce the Tests
154+
## 1: Reproduce the Tests
91155

92156
First, reproduce the **FastPass** tests for A11y using the [Accessibility Insights for Web
93157
][1] browser extension. This extension is designed to help you identify and fix accessibility issues on your website.
94158

95159
![image](https://github.com/BlackPythonDevs/blackpythondevs.github.io/assets/44526468/222e6653-c963-4518-a297-262d656216a7)
96160

97-
### 2: Capture the Issues
161+
## 2: Capture the issues
98162

99-
If the Accessibility Insights for Web extension identifies any issues, capture a screenshot of the issues on the [BPDevs](https://blackpythondevs.github.io/) website. You can do this by pressing <kbd>Ctrl</kbd> + <kbd>Shift</kbd> + <kbd>S</kbd> on your keyboard.
163+
If the Accessibility Insights for Web extension identifies any issues, capture a screenshot of the issues on the [BPDevs](https://BlackPythonDevs.github.io/) website. You can do this by pressing <kbd>Ctrl</kbd> + <kbd>Shift</kbd> + <kbd>S</kbd> on your keyboard.
100164

101-
### 3: Make the Necessary Changes
165+
## 3: Make the Necessary Changes
102166

103167
Next, make the necessary changes to fix the identified issues. This might involve modifying the CSS of the website.
104168

105-
### 4: Test the Changes
169+
## 4: Test the Changes
106170

107171
After making the changes, run the **FastPass** test again and capture a screenshot showing no accessibility issues. This will serve as proof that the issues have been successfully resolved.
108172

109173
![image](https://github.com/BlackPythonDevs/blackpythondevs.github.io/assets/44526468/9a284f43-3cde-4370-9eab-1d302ed65e9e)
110174

111-
### 5: Check and Pass Other Tests
175+
## 5: Check and Pass Other Tests
112176

113177
Finally, check and pass other tests, such as the rules with `pre-commit`. This ensures that your changes are in line with the existing codebase and do not introduce any new issues.
114178

115-
## Translations
179+
# Translations
116180

117181
We welcome translations for the Black Python Devs website in all languages! Here's how you can contribute:
118182

0 commit comments

Comments
 (0)