factorialsucks
factorialsucks copied to clipboard
dockerization, makefile and some fixes
We began to use factorial, to control working hours, and we had to enter the hours from the beginning of the year. To do this, we use this tool, and we dockerize it. We also created some common targets. With this, we were able to enter the hours from any environment and person.
So, here is the PR, in case you want to integrate it.
Thank you very much for the tool. All the best.
Hi @svg153
Thank you for this contribution.
I think I understand the advantages of dockerizing, however I would like to keep the main usage of the app as a standalone/installable binary. I already include binaries for most platforms with each release, have you seen those in the releases page?
In any case, can you please update the Readme file so that anything related to Docker is in a separate section under the current contents? I think that would be enough to merge this.
Hi @alejoar.
I updated the branch with these changes:
- Move Docker information in README.md to a separate section (
Docker
) under the current content. - Create new Makefile targets that use native build.
- Rename Docker Makefile targets.
- Add
factorialsucks
binary created by native build in the root of the repository to.gitignore
.
Thanks.