Skip to content

Latest commit

History

History
243 lines (168 loc) 路 8.71 KB

CONTRIBUTING.md

File metadata and controls

243 lines (168 loc) 路 8.71 KB

Contributing to Ferdi 5

馃帀 First off, thanks for taking the time and your effort to make Ferdi better! 馃帀

Table of contents

Code of Conduct

This project and everyone participating in it is governed by the Ferdi Code of Conduct. By participating, you are expected to uphold this code. Please report unacceptable behavior to hello@getferdi.com.

What should I know before I get started?

For the moment, Ferdi's development is a bit slow but all contributions are highly appreciated. Check this issue for discussion.

How Can I Contribute?

As a basic rule, before filing issues, feature requests or anything else. Take a look at the issues and check if this has not already been reported by another user. If so, engage in the already existing discussion.

Setting up your Development machine

Install System-level dependencies

Node.js, npm, pnpm

Please make sure you are conforming to the engines requirements used by the developers/contributors as specified in the package.json file.

Currently, these are the combinations of system dependencies that work for MacOS/Linux/Windows:

node -v
v14.18.1
npm -v
6.14.13
pnpm -v
6.17.1

Note: You can choose any package manager to manage multiple versions of node and npm. For eg, nvm or asdf.

Git

The version 2.23.0 for Git is working fine for development. You can then use the console from Git to do the development procedure.

Note: This list can likely get outdated. If so, please refer to the specific version of the electronuserland builder that we use in our Dockerfile.

Debian/Ubuntu

apt-get update -y && apt-get install --no-install-recommends -y rpm ruby gem && gem install fpm --no-ri --no-rdoc --no-document

Fedora

dnf install libX11-devel libXext-devel libXScrnSaver-devel libxkbfile-devel rpm

Windows

Please make sure you run this command as an administrator:

npm i -g windows-build-tools --vs2015

node-gyp

We need node-gyp to be able to compile any native dependencies

npm ls -g node-gyp@8.2.0 || npm i -g node-gyp@8.2.0

Clone repository with submodule

git clone https://github.com/getferdi/ferdi.git
cd ferdi
git submodule update --init --recursive

It is important you execute the last command to get the required submodules (recipes).

Local caching of dependencies

Set these env vars into your profile (or if you use direnv, you can manage them via the respective .envrc file)

export ELECTRON_CACHE=$HOME/.cache/electron
export ELECTRON_BUILDER_CACHE=$HOME/.cache/electron-builder

Install dependencies

Run the following command to install all dependencies, and link sibling modules with Ferdi.

npm i

If you encounter the gyp: No Xcode or CLT version error on macOS at this step, please have a look here.

Fix native modules to match current electron node version

npm run build

Package recipe repository

Ferdi requires its recipes to be packaged before it can use it. When running Ferdi as a development instance, you'll need to package the local recipes before you can create any services inside Ferdi.

cd recipes && pnpm i && pnpm run package

Using Docker to build a linux-targetted packaged app

docker build -t ferdi-package-`uname -m` .

The above will place all the built artifacts into the /ferdi folder within the image.

If you want to copy them outside of the image, simply mount a volume into a different location, and copy all files from /ferdi into the mounted folder (/ferdi-out in the example command below).

DATE=`date +"%Y-%b-%d-%H-%M"`
mkdir -p ~/Downloads/$DATE
docker run -e GIT_SHA=`git rev-parse --short HEAD` -v ~/Downloads/$DATE:/ferdi-out -it ferdi-package sh
# inside the container:
mv /ferdi/Ferdi-*.AppImage /ferdi-out/Ferdi-$GIT_SHA.AppImage
mv /ferdi/ferdi-*.tar.gz /ferdi-out/Ferdi-$GIT_SHA.tar.gz
mv /ferdi/ferdi-*.x86_64.rpm /ferdi-out/Ferdi-x86_64-$GIT_SHA.rpm
mv /ferdi/ferdi_*_amd64.deb /ferdi-out/Ferdi-amd64-$GIT_SHA.deb
mv /ferdi/ferdi-*.freebsd /ferdi-out/Ferdi-$GIT_SHA.freebsd
mv /ferdi/ferdi /ferdi-out/Ferdi-$GIT_SHA
mv /ferdi/latest-linux.yml /ferdi-out/latest-linux-$GIT_SHA.yml

Code Signing on a mac

If you are building the packaged app (on a mac) for local testing, you can set this environment variable to bypass the code signing step during the packaging process (npm run build):

export CSC_IDENTITY_AUTO_DISCOVERY=false

Or else, if you want to self-sign on a mac with non-registered certificate (not for distribution of the resulting package), you can follow this thread and run this command:

codesign --deep --force --verbose --sign - node_modules/electron/dist/Electron.app

Start development app

Run these two commands simultaneously in different terminals:

npm run dev
DEBUG=Ferdi:* npm run start
  • Optionally, you can run both commands in one terminal with concurrently:
DEBUG_COLORS=1 DEBUG=Ferdi:* npm run start:all-dev

Note: please prefer debug() over console.log().

Styleguide

Git Commit Messages format

  • Use the present tense ("Add feature" not "Added feature")
  • Use the imperative mood ("Move cursor to..." not "Moves cursor to...")
  • Limit the first line to 72 characters or less
  • Reference issues and pull requests liberally after the first line
  • When only changing documentation, include [ci skip] in the commit description

Javascript Coding style-checker

  • Please use prettier and the defined rules to maintain a consistent style

Packaging

npm run build

Assets will be available in the out folder.

Release

git checkout nightly && git pull -r
git checkout release
git merge --no-ff nightly --no-verify
# <manually resolve conflicts>
# <manually bump version with 'beta' name (if beta) in `package.json` and `package-lock.json`>
# <create commit>
# <create tag>
git push

This will automatically trigger the build, as part of which, a new, draft release will be created here. Once all the assets are uploaded (19 assets in total), publish the release (you will need elevated permissions in GitHub for doing this). The last commit of the release branch will be tagged.

Updating the code after a hiatus

Since we are making a lot of changes that involve restructuring the code as well as taking a hard look at the dependencies (including node versions), please remember to update them by running the appropriate command of your chosen package manager.