]> git.p6c8.net - jirafeau_project.git/blobdiff - CONTRIBUTING.md
Merge branch 'bug_content_length' into 'next-release'
[jirafeau_project.git] / CONTRIBUTING.md
index 7c5ed44acfa6bb503bcaeae94dd938b6eeae2c79..c80c1cddf5054a5e78fb921f1cae0fbf45226bd9 100644 (file)
@@ -4,6 +4,8 @@ Hi,
 
 this document is made for newcomers in Jirafeau who are digging into the code.
 
+If you have further questions, then just ask for help ๐Ÿค“.
+
 ## General principle
 
 Jirafeau is made in the [KISS](http://en.wikipedia.org/wiki/KISS_principle) way (Keep It Simple, Stupid).
@@ -12,16 +14,17 @@ It is meant to be a simple filehosting service, simple to use, simple to install
 
 This project won't evolve to a file manager and will focus to keep a very few dependencies.
 
-So things like a markdown parser for the ToS or E-Mail tasks would be usefull for sure, but may be [rejected](https://gitlab.com/mojo42/Jirafeau/issues/37#note_1191566) since they would a lot of dependencies and makes the project more complex.
+So things like a markdown parser for the ToS or E-Mail tasks would be useful for sure, but were [rejected](https://gitlab.com/mojo42/Jirafeau/issues/37#note_1191566) long ago since they would add a lot of dependencies and makes the project more complex.
 
 ## Structure
 
-Here is a little explaination of Jirafeau's arboresence in a simplified
-view only to show the most importants files and their role.
+Here is a little explanation of Jirafeau's structure in a simplified
+view only to show the most important files and their role.
 
 ```
 .
 โ”œโ”€โ”€ admin.php : administration interface to manage links and files
+โ”œโ”€โ”€ docker : folder containing some configuration files to run Jirafeau in docker
 โ”œโ”€โ”€ f.php : permits to download files or show the download page
 โ”œโ”€โ”€ index.php : provides a web interface to interact with API
 โ”œโ”€โ”€ script.php : API interface (all file actions happen here - upload, deletion, etc)
@@ -29,29 +32,29 @@ view only to show the most importants files and their role.
 โ”œโ”€โ”€ tos.php : "Terms of Service" page
 โ”œโ”€โ”€ lib
 โ”‚ย ย  โ”œโ”€โ”€ config.original.php : default parameters
-โ”‚ย ย  โ”œโ”€โ”€ config.local.php : the users parameters (auto generated, not versionized)
+โ”‚ย ย  โ”œโ”€โ”€ config.local.php : the users parameters (auto generated, not versioned)
 โ”‚ย ย  โ”œโ”€โ”€ functions_*.js : JavaScript functions for index.php (AJAX etc)
 โ”‚ย ย  โ”œโ”€โ”€ functions.php : core functions and tools of Jirafeau
 โ”‚ย ย  โ”œโ”€โ”€ tos.original.txt : default text show on the ToS page
-โ”‚ย ย  โ”œโ”€โ”€ tos.local.txt : a users alternative text show on the ToS page (not versionized)
+โ”‚ย ย  โ”œโ”€โ”€ tos.local.txt : a users alternative text show on the ToS page (not versioned)
 โ”‚ย ย  โ”œโ”€โ”€ settings.php : core settings of Jirafeau, includes the configuration params automatically
 โ”‚ย ย  โ”œโ”€โ”€ locales : language folder, contains all language files
 โ”‚ย ย  โ””โ”€โ”€ template
 โ”‚ย ย      โ”œโ”€โ”€ footer.php : footer with links to source and ToS for all HTML views
 โ”‚ย ย      โ””โ”€โ”€ header.php : header with logo and title for all HTML views
 โ”œโ”€โ”€ media : folder containing all skins
-โ””โ”€โ”€ var-xxxxxxx : the users folder containing all data (auto generated, not versionized)
-    โ”œโ”€โ”€ async : chunks of uploaded files (not succressfull yet) 
+โ””โ”€โ”€ var-xxxxxxx : the users folder containing all data (auto generated, not versioned)
+    โ”œโ”€โ”€ async : chunks of uploaded files (not successful yet) 
     โ”œโ”€โ”€ files : all files that have been uploaded successfully
-        โ”œโ”€โ”€ [hashed file name] : the original file
-        โ”œโ”€โ”€ [hashed file name]_count : count many links to this file exist
-    โ””โ”€โ”€ links : all links, including meta-informations, pointing to files
-        รข\94\9cโ”€โ”€ [link] : the link file, includes which original file should be used and some meta data like creation date, expiration time
+    โ”‚   โ”œโ”€โ”€ [hashed file name] : the original file
+    โ”‚   โ””โ”€โ”€ [hashed file name]_count : count many links to this file exist
+    โ””โ”€โ”€ links : all links, including metadata, pointing to files
+        รข\94\94โ”€โ”€ [link] : the link file, includes which original file should be used and some meta data like creation date, expiration time
 ```
 
 ## Translations
 
-Translation may be add via [Jirafeau's Weblate](https://hosted.weblate.org/projects/jirafeau/master/).
+Translation may be added via [Jirafeau's Weblate](https://hosted.weblate.org/projects/jirafeau/master/).
 
 ## Coding style
 
@@ -60,14 +63,43 @@ Translation may be add via [Jirafeau's Weblate](https://hosted.weblate.org/proje
 
 ## Branches
 
-* ```master``` = latest release, e.g. 2.0.1
-* ```next-release``` = development branch - all new features are merged into this branch until the next version is released. So use this branch as base while developing new features or bugfixes.
-* ```test``` = sandbox branch to test new features or merge requests, or run integration tests. The content of this branch may change at any time.
+* `master` = latest release, e.g. 2.0.1
+* `next-release` = development branch - all new features are merged into this branch until the next version is released. So use this branch as base while developing new features or bugfixes.
+* `test` = sandbox branch to test new features or merge requests, or run integration tests. The content of this branch may change at any time.
 
 ## Merge Requests
 
 Please create one branch for each feature and send one merge request for each branch. 
 
-Dont squash several changes or commits into one merge request as this is hard to review.
-
-Please use ```next-release``` as base branch and send your merge request to this branch (not ```master```).
+Don't squash several changes or commits into one merge request as this is hard to review.
+
+Please use `next-release` as base branch and send your merge request to this branch (not `master`).
+
+Quick walkthrough:
+
+* Create ticket for new feature
+* Fork the original repository, clone the own repository, add the original repository as upstream
+* Checkout `next-release` branch `git checkout next-release`
+* Create a new branch on top of that one, e.g. `some-feature`: `git checkout -b some-feature`
+* Make your change
+* You may check if the project is still [REUSE Compliant](https://reuse.software/) by running `docker run -v $(pwd):/code --rm fsfe/reuse /bin/sh -c "cd /code && reuse lint"`
+* Commit changes โ†’ push โ†’ send merge request `git add -A; git commit; git push` MR via GitLab (link shown in console)
+* Feature is reviewed
+  * MR accepted: Reviewer checks out `next-release` branch and cherry-picks the commit `git checkout next-release; git cherry-pick be4369641; git push`
+  * MR declined: Reviewer add some notes, Developer rebases his branch, adds neccessary changes, force pushes the branch, ask a reviewer to review the changes in the merge request ticket (as Gitlab recognizes them automatically) `git checkout some-feature; git rebase upstream/next-release` โ€ฆ[add changes]โ€ฆ `git add -A, git commit --amend; git push -f`
+
+## New Releases
+
+* Fetch weblate and rebase and import translations
+* If the release is not done for security purposes: create a new issue and freeze next-release branch for at least week.
+* Compare the [`next-release` branch to `master`](https://gitlab.com/jirafeau/Jirafeau/compare/master...next-release)
+* Add a list of noteworthy features and bugfixes to `CHANGELOG.md`
+* Add eventual upgrade procedure to `CHANGELOG.md`. Make sure to list all new configuration items.
+* Build and test docker image
+* Change the version, using [semantic versioning](http://semver.org/), in `settings.php`
+* Merge `next-release` branch to `master`
+* Tag the `master` with the new version
+* Push branch and tag
+* Push new docker image
+* Update the demo page
+* Dance a little

patrick-canterino.de