209 lines
6.3 KiB
Markdown
209 lines
6.3 KiB
Markdown
# Contributing to Cal.com
|
|
|
|
Contributions are what makes the open source community such an amazing place to learn, inspire, and create. Any contributions you make are **greatly appreciated**.
|
|
|
|
- Before jumping into a PR be sure to search [existing PRs](https://github.com/calcom/cal.com/pulls) or [issues](https://github.com/calcom/cal.com/issues) for an open or closed item that relates to your submission.
|
|
|
|
## Priorities
|
|
|
|
<table>
|
|
<tr>
|
|
<td>
|
|
Type of Issue
|
|
</td>
|
|
<td>
|
|
Priority
|
|
</td>
|
|
</tr>
|
|
<tr>
|
|
<td>
|
|
Minor improvements, non-core feature requests
|
|
</td>
|
|
<td>
|
|
<a href="https://github.com/calcom/cal.com/issues?q=is:issue+is:open+sort:updated-desc+label:%22Low+priority%22">
|
|
<img src="https://img.shields.io/badge/-Low%20Priority-green">
|
|
</a>
|
|
</td>
|
|
</tr>
|
|
<tr>
|
|
<td>
|
|
Confusing UX (... but working)
|
|
</td>
|
|
<td>
|
|
<a href="https://github.com/calcom/cal.com/issues?q=is:issue+is:open+sort:updated-desc+label:%22Medium+priority%22">
|
|
<img src="https://img.shields.io/badge/-Medium%20Priority-yellow">
|
|
</a>
|
|
</td>
|
|
</tr>
|
|
<tr>
|
|
<td>
|
|
Core Features (Booking page, availability, timezone calculation)
|
|
</td>
|
|
<td>
|
|
<a href="https://github.com/calcom/cal.com/issues?q=is:issue+is:open+sort:updated-desc+label:%22High+priority%22">
|
|
<img src="https://img.shields.io/badge/-High%20Priority-orange">
|
|
</a>
|
|
</td>
|
|
</tr>
|
|
<tr>
|
|
<td>
|
|
Core Bugs (Login, Booking page, Emails are not working)
|
|
</td>
|
|
<td>
|
|
<a href="https://github.com/calcom/cal.com/issues?q=is:issue+is:open+sort:updated-desc+label:Urgent">
|
|
<img src="https://img.shields.io/badge/-Urgent-red">
|
|
</a>
|
|
</td>
|
|
</tr>
|
|
</table>
|
|
|
|
## Developing
|
|
|
|
The development branch is `main`. This is the branch that all pull
|
|
requests should be made against. The changes on the `main`
|
|
branch are tagged into a release monthly.
|
|
|
|
To develop locally:
|
|
|
|
1. [Fork](https://help.github.com/articles/fork-a-repo/) this repository to your
|
|
own GitHub account and then
|
|
[clone](https://help.github.com/articles/cloning-a-repository/) it to your local device.
|
|
2. Create a new branch:
|
|
|
|
```sh
|
|
git checkout -b MY_BRANCH_NAME
|
|
```
|
|
|
|
3. Install yarn:
|
|
|
|
```sh
|
|
npm install -g yarn
|
|
```
|
|
|
|
4. Install the dependencies with:
|
|
|
|
```sh
|
|
yarn
|
|
```
|
|
|
|
5. Set up your `.env` file:
|
|
|
|
- Duplicate `.env.example` to `.env`.
|
|
- Use `openssl rand -base64 32` to generate a key and add it under `NEXTAUTH_SECRET` in the `.env` file.
|
|
- Use `openssl rand -base64 24` to generate a key and add it under `CALENDSO_ENCRYPTION_KEY` in the `.env` file.
|
|
|
|
6. Setup Node
|
|
If your Node version does not meet the project's requirements as instructed by the docs, "nvm" (Node Version Manager) allows using Node at the version required by the project:
|
|
|
|
```sh
|
|
nvm use
|
|
```
|
|
|
|
You first might need to install the specific version and then use it:
|
|
|
|
```sh
|
|
nvm install && nvm use
|
|
```
|
|
|
|
You can install nvm from [here](https://github.com/nvm-sh/nvm).
|
|
|
|
7. Start developing and watch for code changes:
|
|
|
|
```sh
|
|
yarn dev
|
|
```
|
|
|
|
## Building
|
|
|
|
You can build the project with:
|
|
|
|
```bash
|
|
yarn build
|
|
```
|
|
|
|
Please be sure that you can make a full production build before pushing code.
|
|
|
|
## Testing
|
|
|
|
More info on how to add new tests coming soon.
|
|
|
|
### Running tests
|
|
|
|
This will run and test all flows in multiple Chromium windows to verify that no critical flow breaks:
|
|
|
|
```sh
|
|
yarn test-e2e
|
|
```
|
|
|
|
#### Resolving issues
|
|
|
|
##### E2E test browsers not installed
|
|
|
|
Run `npx playwright install` to download test browsers and resolve the error below when running `yarn test-e2e`:
|
|
|
|
```
|
|
Executable doesn't exist at /Users/alice/Library/Caches/ms-playwright/chromium-1048/chrome-mac/Chromium.app/Contents/MacOS/Chromium
|
|
```
|
|
|
|
## Linting
|
|
|
|
To check the formatting of your code:
|
|
|
|
```sh
|
|
yarn lint
|
|
```
|
|
|
|
If you get errors, be sure to fix them before committing.
|
|
|
|
## Making a Pull Request
|
|
|
|
- Be sure to [check the "Allow edits from maintainers" option](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork) while creating your PR.
|
|
- If your PR refers to or fixes an issue, be sure to add `refs #XXX` or `fixes #XXX` to the PR description. Replacing `XXX` with the respective issue number. See more about [Linking a pull request to an issue](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue).
|
|
- Be sure to fill the PR Template accordingly.
|
|
- Review [App Contribution Guidelines](./packages/app-store/CONTRIBUTING.md) when building integrations
|
|
|
|
## Guidelines for committing yarn lockfile
|
|
|
|
Do not commit your `yarn.lock` unless you've made changes to the `package.json`. If you've already committed `yarn.lock` unintentionally, follow these steps to undo:
|
|
|
|
If your last commit has the `yarn.lock` file alongside other files and you only wish to uncommit the `yarn.lock`:
|
|
```bash
|
|
git checkout HEAD~1 yarn.lock
|
|
git commit -m "Revert yarn.lock changes"
|
|
```
|
|
If you've pushed the commit with the `yarn.lock`:
|
|
1. Correct the commit locally using the above method.
|
|
2. Carefully force push:
|
|
|
|
```bash
|
|
git push origin <your-branch-name> --force
|
|
```
|
|
|
|
If `yarn.lock` was committed a while ago and there have been several commits since, you can use the following steps to revert just the `yarn.lock` changes without impacting the subsequent changes:
|
|
|
|
1. **Checkout a Previous Version**:
|
|
- Find the commit hash before the `yarn.lock` was unintentionally committed. You can do this by viewing the Git log:
|
|
```bash
|
|
git log yarn.lock
|
|
```
|
|
- Once you have identified the commit hash, use it to checkout the previous version of `yarn.lock`:
|
|
```bash
|
|
git checkout <commit_hash> yarn.lock
|
|
```
|
|
|
|
2. **Commit the Reverted Version**:
|
|
- After checking out the previous version of the `yarn.lock`, commit this change:
|
|
```bash
|
|
git commit -m "Revert yarn.lock to its state before unintended changes"
|
|
```
|
|
|
|
3. **Proceed with Caution**:
|
|
- If you need to push this change, first pull the latest changes from your remote branch to ensure you're not overwriting other recent changes:
|
|
```bash
|
|
git pull origin <your-branch-name>
|
|
```
|
|
- Then push the updated branch:
|
|
```bash
|
|
git push origin <your-branch-name>
|
|
```
|