Skip to content

Commit 7ad9c43

Browse files
committed
Rename CONTRIBUTING.md to DEVELOPING.md
1 parent 35ccfa1 commit 7ad9c43

1 file changed

Lines changed: 8 additions & 22 deletions

File tree

CONTRIBUTING.md renamed to DEVELOPING.md

Lines changed: 8 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,18 @@
1-
# Contributing
1+
# Developing
22

3-
This file will guide you through the process of getting to project up and running, in case you want to provide coding contributions.
3+
This file will guide you through the process of getting to project up and running, in case you want to work on it locally.
44

5-
You will also see how to ensure the code fulfills the expected code checks, and how to create a pull request.
5+
You will also see how to ensure the code fulfills the expected code checks, and verify it meets expected quality.
66

77
## System dependencies
88

99
The project provides all its dependencies as docker containers through a `docker compose` configuration.
1010

11-
Because of this, the only actual dependencies are [docker](https://docs.docker.com/get-docker/) and [docker compose](https://docs.docker.com/compose/install/).
11+
Because of this, the only actual dependency is [docker](https://docs.docker.com/get-docker/).
1212

1313
## Setting up the project
1414

15-
The first thing you need to do is fork the repository, and clone it in your local machine.
15+
The first thing you need to do is clone the repository in your local machine.
1616

1717
Then you will have to follow these steps:
1818

@@ -22,7 +22,7 @@ Then you will have to follow these steps:
2222
cp config/params/shlink_dev_env.php.dist config/params/shlink_dev_env.php
2323
```
2424
25-
The `shlink_dev_env.php` file is gitignored, so you can customize it as you want. For example, by adding your own GeoLite license key.
25+
The `shlink_dev_env.php` file is git-ignored, so you can customize it as you want. For example, by adding your own GeoLite license key.
2626
2727
* Start-up the project by running `docker compose up`.
2828
@@ -34,9 +34,9 @@ Then you will have to follow these steps:
3434
* Run `./indocker bin/cli db:migrate` to get database migrations up to date.
3535
* Run `./indocker bin/cli api-key:generate` to get your first API key generated.
3636
37-
Once you finish this, you will have the project exposed in ports `8800` through RoadRunner and `8000` through nginx+php-fpm.
37+
Once you finish this, you will have the project exposed in ports `8800` through RoadRunner, `8008` through FrankenPHP and `8000` through nginx+php-fpm.
3838
39-
> Note: The `indocker` shell script is a helper tool used to run commands inside the main docker container.
39+
> Note: The `indocker` shell script is a helper tool used to run commands inside the main RoadRunner docker container.
4040
4141
## Project structure
4242
@@ -111,8 +111,6 @@ In order to ensure stability and no regressions are introduced while developing
111111
112112
They use Maria DB as the database engine, and include the same fixtures as the API tests, that ensure the same data exists at the beginning of the execution.
113113
114-
Depending on the kind of contribution, maybe not all kinds of tests are needed, but the more you provide, the better.
115-
116114
## Running code checks
117115
118116
* Run `./indocker composer cs` to check coding styles are fulfilled.
@@ -135,18 +133,6 @@ The project provides a Swagger UI container for dev envs, which can be accessed
135133
136134
It will automatically load the contents of `docs/swagger`, so you can make any updates and they will get reflected.
137135
138-
## Pull request process
139-
140-
**Important!**: Before starting to work on a pull request, make sure you always [open an issue](https://github.com/shlinkio/shlink/issues/new/choose) first.
141-
142-
This is important because any contribution needs to be discussed first. Maybe there's someone else already working on something similar, or there are other considerations to have in mind.
143-
144-
Once everything is clear, to provide a pull request to this project, you should always start by creating a new branch, where you will make all desired changes.
145-
146-
The base branch should always be `develop`, and the target branch for the pull request should also be `develop`.
147-
148-
Before your branch can be merged, all the checks described in [Running code checks](#running-code-checks) have to be passing. You can verify that manually by running `./indocker composer ci`, or wait for the build to be run automatically after the pull request is created.
149-
150136
## Architectural Decision Records
151137
152138
The project includes logs for some architectural decisions, using the [adr](https://adr.github.io/) proposal.

0 commit comments

Comments
 (0)