Chapter 16: Polish Your Code And Submit Your PR

In the previous chapters we saw how to create a fully functional business application. Now we will prepare our code to be shared and learn how to share it.

Coding guidelines

We will start refactoring the code to match to the Flectra coding guidelines. The guidelines aim to improve the quality of the Flectra Apps code.

Reference: you will find the Flectra coding guidelines in Coding guidelines.


Polish your code.

Refactor your code to respect the coding guidelines. Don’t forget to run your linter and respect the module structure, the variable names, the method name convention, the model attribute order and the xml ids.

Your first Pull Request (PR)

The following is intended to be used by Flectra staff members only. All repositories mentioned are not accessible for third-parties.

Now that your code respects the Flectra code guidelines and is polished up, let’s share it with others.

Configure development repository

We will start by creating a git development environment for the ‘custom’ folder, like we did in the beginning for ‘flectra’ and ‘professional’. The PR will target the flectra/technical-training-sandbox repository.

To do this the first step is to fork in GitHub this repository to create your own development repository.

After you successfully created your development repository, we will configure your existing $HOME/src/custom folder to be able to commit your work.

$ cd $HOME/src/custom
$ git init
$ git remote add training
$ git remote add training-dev #Don't forget to change xyz-flectra to your own GitHub account
$ git remote set-url --push training no_push #Avoid pushing directly into the main repository
$ git fetch training master
$ git branch master training/master
$ git checkout master #Two new files will be downloaded and .gitignore

Note that if your work is already in a Git and GitHub configured folder the previous steps are not needed. This is the case when you start working in the ‘flectra’ or ‘professional’ folders.

Branch, Commit & Push

Before creating the PR, a new working branch has to be created and used to commit the code. Afterwards the branch will be pushed to the development repository.

Create your working branch:

$ git checkout -b master-my_first_branch-xyz

Your branch name must follow the following name structure : <targetVersion>-<feature>-<trigram>

Example: The branch master-sale-fixes-abc on flectra-dev/flectra is a branch containing fixes for the sales app in the flectra/flectra repository, to be deployed in master and done by ABC.

Commit your code:

$ git add .
$ git commit

Everyone reads your commit messages!

The commit message is very important, follow the Git guidelines.

Push your new branch to your development repository:

$ git push -u training-dev master-my_first_branch-xyz


NEVER use git push --force (or git push -f) with a branch that does not belong to you or in a stable branch. Never, never, never!


Remember that you can use git status at any point to check the current status of your branch


You can create git aliases to shorten some of these commands. Ask your colleagues if they use git aliases and get inspired by their work.

Create your first PR

After pushing your branch to your development repository you will see an output similar to:

Enumerating objects: 24, done.
Counting objects: 100% (24/24), done.
Compressing objects: 100% (20/20), done.
Writing objects: 100% (23/23), 6.67 KiB | 525.00 KiB/s, done.
Total 23 (delta 1), reused 0 (delta 0)
remote: Resolving deltas: 100% (1/1), done.
remote: Create a pull request for 'master-my_first_branch-xyz' on GitHub by visiting:
* [new branch]      master-my_first_branch-xyz -> master-my_first_branch-xyz
Branch 'master-my_first_branch-xyz' set up to track remote branch 'master-my_first_branch-xyz' from 'training-dev'.

There are two ways to create the PR:

  • Click on the link displayed in the output of the push command.

  • Open a browser in your development repository There will be a button to create a new pull request.


You will notice that your commit message is used as the pull request message. This occurs if you have only 1 commit. If you have multiple commits, you can either make a summary of the commits as the PR message or if there aren’t that many commits you can just copy/paste your commit messages as the PR message.


Test on the runbot

Flectra has its own CI server named runbot. All commits, branches and PR will be tested to avoid regressions or breaking of the stable versions. All the runs that pass the tests are deployed on their own server with demo data.


Play with the runbot.

Feel free to go to the runbot website and open the last stable version of Flectra to check out all the available applications and functionalities.