Skip to content

Commit 08ff4b7

Browse files
committed
updated contribute guide
1 parent 4632005 commit 08ff4b7

1 file changed

Lines changed: 45 additions & 17 deletions

File tree

CONTRIBUTING.md

Lines changed: 45 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,40 @@
1-
### **How to Contribute to the Papers Repository**
1+
# Contribute to the Papers Repository
22

33
We appreciate your interest in contributing to the `papers-codechef` repository! Please follow these guidelines to ensure a smooth and effective contribution process.
44

55
---
66

7-
### **1. Get an Issue Assigned**
7+
## Contribution ideas
88

9-
All contributions must be tied to an issue. Go to the issues page at `https://github.com/CodeChefVIT/papers-codechef/issues`
10-
You can either ask a board member to assign you an existing issue or create a new issue and ask a board member to assign it to you.
9+
If you're looking for ideas about what to work on, check out:
10+
11+
- Our [issues](https://github.com/CodeChefVIT/papers-codechef/issues) based on votes by the community.
1112

1213
---
1314

14-
### **2. Fork the Repository**
15+
## How to Contribute
16+
17+
### 1. Get an Issue Assigned
18+
19+
To start working on an issue you self assign yourself to the issue, and make sure you mention the issue you are solving in the PR you make
20+
21+
---
22+
23+
### 2. Fork the Repository
1524

1625
To start working, you need to create your own copy of the repository.
1726

1827
- **Fork the repo:** Navigate to `https://github.com/CodeChefVIT/papers-codechef` and click the "Fork" button to create a copy of the repository on your account. This allows you to freely experiment with changes without affecting the original project.
19-
28+
2029
<img width="1480" height="327" alt="image" src="https://github.com/user-attachments/assets/3df5e7e3-ebad-4598-b51f-0fa64f0f1cca" />
21-
30+
2231
- **Create the fork:** When prompted, make sure the "Copy the `prod` branch only" option is unchecked.
23-
32+
2433
<img width="1376" height="685" alt="image" src="https://github.com/user-attachments/assets/035dc07b-3296-4706-8037-f03de90fdafb" />
2534

2635
---
2736

28-
### **3. Clone Your Fork**
37+
### 3. Clone Your Fork
2938

3039
After forking, you need to get the code onto your local machine.
3140

@@ -34,26 +43,26 @@ After forking, you need to get the code onto your local machine.
3443
<img width="1486" height="1025" alt="image" src="https://github.com/user-attachments/assets/61a1fb0d-9000-49c0-be8a-7fccce966963" />
3544

3645
- **Clone the repo:** Copy the URL of your fork to your clipboard.
37-
- **Run `git clone {url}`:** This command will clone the repository. You can use either **HTTPS** (less secure, easier) or **SSH** (more secure, needs a `.ssh` file setup). If you're unsure which method to use, ask a board member for help.
46+
- **Run `git clone {url}`:** This command will clone the repository. You can use either **HTTPS** (less secure, easier) or **SSH** (more secure, needs a `.ssh` file setup).
3847

3948
<img width="1490" height="1045" alt="image" src="https://github.com/user-attachments/assets/49ff2344-25b1-49d2-bb96-8a9581b0c73a" />
4049

4150
---
4251

43-
### **4. Set Up Your Local Environment**
52+
### 4. Set Up Your Local Environment
4453

4554
To get the project running, you'll need to set up your local environment.
4655

47-
- **Create a `.env` file:** Create a new file named `.env` and ask a board member for its contents.
56+
- **Create a `.env` file:** Create a new file named `.env` and use the .env.example file to create your own .env file and put in your your own environment variables to make the project functional.
4857
- **Install dependencies:** Run `pnpm i` in your terminal to install all necessary dependencies.
49-
- **Do git checkout staging** if you want to contribute during a major revamp that is going on. Ask a board which branch to take as a base, if unsure.
58+
- **Do git checkout staging** if you want to contribute during a major revamp that is going on.
5059
- **Run the project:** Run `pnpm dev` to start the project.
5160

5261
<img width="1600" height="969" alt="image" src="https://github.com/user-attachments/assets/74ef04d2-fb5e-4b80-b014-f9207fc01714" />
5362

5463
---
5564

56-
### **5. Work on Your Code**
65+
### 5. Work on Your Code
5766

5867
Once your environment is set up, you're ready to start coding.
5968

@@ -76,14 +85,33 @@ Once your environment is set up, you're ready to start coding.
7685

7786
---
7887

79-
### **6. Submit a Pull Request**
88+
### 6. Submit a Pull Request
8089

8190
- **Open a Pull Request:** On your GitHub page for the forked repository, you will see a button to "Open pull request" since your branch is ahead of the upstream. Click this to begin the process.
8291

8392
<img width="1482" height="1052" alt="image" src="https://github.com/user-attachments/assets/b72f3735-5801-496b-a59b-6f184c39eaa6" />
8493

8594
- **Target the `staging` branch:** When creating the pull request, make sure you set the **base branch** to `staging`.
86-
95+
8796
<img width="1482" height="1052" alt="image" src="https://github.com/user-attachments/assets/7578cf06-160e-4965-9273-5b68e926911d" />
8897

89-
- **Request a review:** After creating the pull request, direct message a board member to review your PR. Any new changes you push to your branch will be automatically attached to the pull request.
98+
- **Request a review:** After creating the pull request, wait for a organization member to review your PR. Any new changes you push to your branch will be automatically attached to the pull request.
99+
100+
---
101+
102+
## Mandatory PR contents
103+
104+
Please ensure that any PR you make contains these things -
105+
106+
- Purpose and issue which the PR is made regarding.
107+
- Showcase the changes made by attaching screenshots of the visual changes done.
108+
- List the major changes made in this PR.
109+
- Mention any bug fixes, known issues, or follow-ups needed.
110+
111+
**Important:** Make sure there are no merge conflicts with the branch while making a PR otherwise it will not be merged, run `pnpm build` before making a PR to ensure there are no build errors while merging as well.
112+
113+
## Tips to improve the chances of your PR getting reviewed and merged
114+
115+
- Small, focused, incremental pull requests are much easier to review
116+
- Spend time explaining your changes in the pull request body
117+
- Low effort PRs, such as those that just re-arrange syntax, won't be merged without a compelling justification

0 commit comments

Comments
 (0)