View Source Contributing to TypedStruct
TypedStruct is written in Elixir.
For branching management, this project uses
git-flow. The main
branch is
reserved for releases: the development process occurs on develop
and feature
branches. Please never commit to main
.
You can easily set up a development environment featuring all the dependencies,
including Elixir and git-flow
, by using Nix. This is
detailed below.
Setup
Local repository
Fork the repository.
Clone your fork to a local repository:
$ git clone https://github.com/you/typedstruct.git $ cd typedstruct
Add the main repository as a remote:
$ git remote add upstream https://github.com/ejpcmac/typedstruct.git
Checkout
develop
:$ git checkout develop
Development environment (without Nix)
Install an Elixir environment, and optionally install git-flow
.
Development environment (with Nix)
Install Nix by running the script and following the instructions:
$ curl https://nixos.org/nix/install | sh
Optionally install direnv to automatically setup the environment when you enter the project directory:
$ nix-env -i direnv
In this case, you also need to add to your
~/.<shell>rc
:eval "$(direnv hook <shell>)"
Make sure to replace
<shell>
by your shell, namelybash
,zsh
, …In the project directory, if you did not install direnv, start a Nix shell:
$ cd typedstruct $ nix-shell
If you opted to use direnv, please allow the
.envrc
instead of running a Nix shell manually:$ cd typedstruct $ direnv allow
In this case, direnv will automatically update your environment to behave like a Nix shell whenever you enter the project directory.
Git-flow
If you want to use git-flow
and use the standard project configuration, please
run:
$ ./.gitsetup
Building the project
Fetch the project dependencies:
$ cd typedstruct $ mix deps.get
Run the static analyzers:
$ mix check
All the tests should pass.
Workflow
To make a change, please use this workflow:
Checkout
develop
and apply the last upstream changes (use rebase, not merge!):$ git checkout develop $ git fetch --all --prune $ git rebase upstream/develop
For a tiny patch, create a new branch with an explicit name:
$ git checkout -b <my_branch>
Alternatively, if you are working on a feature which would need more work, you can create a feature branch with
git-flow
:$ git flow feature start <my_feature>
Note: always open an issue and ask before starting a big feature, to avoid it not beeing merged and your time lost.
Work on your feature (don’t forget to write typespecs and tests; you can check your coverage with
mix coveralls.html
and opencover/excoveralls.html
):# Some work $ git commit -am "feat: my first change" # Some work $ git commit -am "refactor: my second change" ...
When your feature is ready, feel free to use interactive rebase so your history looks clean and is easy to follow. Then, apply the last upstream changes on
develop
to prepare integration:$ git checkout develop $ git fetch --all --prune $ git rebase upstream/develop
If there were commits on
develop
since the beginning of your feature branch, integrate them by rebasing if your branch has few commits, or merging if you had a long-lived branch:$ git checkout <my_feature_branch> $ git rebase develop
Note: the only case you should merge is when you are working on a big feature. If it is the case, we should have discussed this before as stated above.
Run the tests and static analyzers to ensure there is no regression and all works as expected:
$ mix check
If it’s all good, open a pull request to merge your branch into the
develop
branch on the main repository.
Coding style
Please format your code with mix format
or your editor and follow
this style guide.
All contributed code must be documented and functions must have typespecs. In general, take your inspiration from the existing code.
Please name your commits using Conventional Commits.