]> git.p6c8.net - jirafeau.git/blobdiff - CONTRIBUTING.md
[FEATURE] Add walktrough to create merge requests
[jirafeau.git] / CONTRIBUTING.md
index 67bd4b966cc416495284fbb7e977f564de37b538..f7f21ed79399c5b92d03ca563031c924057059dc 100644 (file)
@@ -1,39 +1,86 @@
+# Contributing
+
 Hi,
 
-This document is only made for newcomers in Jirafeau who are digging into
-the code.
+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).
+
+It is meant to be a simple filehosting service, simple to use, simple to install, simple to maintain.
+
+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.
+
+## Structure
 
 Here is a little explaination of Jirafeau's arboresence in a simplified
 view only to show the most importants files and their role.
 
+```
 .
-β”œβ”€β”€ admin.php : adminitration interface, also permits to download files
+β”œβ”€β”€ admin.php : administration interface to manage links and files
 β”œβ”€β”€ f.php : permits to download files or show the download page
-β”œβ”€β”€ index.php : only provide a html/javascript client to interact with API
-β”œβ”€β”€ script.php : API interface and it's html documentation
+β”œβ”€β”€ index.php : provides a web interface to interact with API
+β”œβ”€β”€ script.php : API interface (all file actions happen here - upload, deletion, etc)
 β”œβ”€β”€ install.php : installation script
-β”œβ”€β”€ tos.php : terms of use the user may edit
+β”œβ”€β”€ tos.php : "Terms of Service" page
 β”œβ”€β”€ lib
-β”‚Β Β  β”œβ”€β”€ config.local.php : user's parameters
-β”‚Β Β  β”œβ”€β”€ config.original.php : default parameters with their documentation
-β”‚Β Β  β”œβ”€β”€ functions_*.js : javascript functions for html/javascript client
-β”‚Β Β  β”œβ”€β”€ functions.php : core functions and tools of jirafeau
-β”‚Β Β  β”œβ”€β”€ locales : langage folder, contain all langage files
+β”‚Β Β  β”œβ”€β”€ config.original.php : default parameters
+β”‚Β Β  β”œβ”€β”€ config.local.php : the users parameters (auto generated, not versionized)
+β”‚Β Β  β”œβ”€β”€ 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)
+β”‚Β Β  β”œβ”€β”€ settings.php : core settings of Jirafeau, includes the configuration params automatically
+β”‚Β Β  β”œβ”€β”€ locales : language folder, contains all language files
 β”‚Β Β  β””── template
-β”‚Β Β      β”œβ”€β”€ footer.php
-β”‚Β Β      β””── header.php
+β”‚Β Β      β”œβ”€β”€ 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 : folder containing all data
-    β”œβ”€β”€ async : chunks of uploaded files
-    β”œβ”€β”€ files : all files that has been successfully uploaded
-    β””── links : all links pointing to files with meta-informations
+└── var-xxxxxxx : the users folder containing all data (auto generated, not versionized)
+    β”œβ”€β”€ async : chunks of uploaded files (not succressfull 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
+        β”œβ”€β”€ [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/).
+
+## Coding style
+
+- This project follows the [PSR-2](http://www.php-fig.org/psr/psr-2/) Coding Style
+- Files must be in UTF-8 without BOM and use Unix Line Endings (LF)
+
+## 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.
+
+## Merge Requests
+
+Please create one branch for each feature and send one merge request for each branch. 
 
-Coding style:
+Dont squash several changes or commits into one merge request as this is hard to review.
 
-- PHP function keywords are alone on a line
-- Braces "{" must are put in a new line
-- Files must be in UTF-8 (without BOM)
-- Uses LF (\n) for end of lines
+Please use ```next-release``` as base branch and send your merge request to this branch (not ```master```).
 
-The whole project is not clean about that, feel free to fix :)
+Quick walktrough:
 
+* 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```
+* 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```

patrick-canterino.de