factorialsucks icon indicating copy to clipboard operation
factorialsucks copied to clipboard

dockerization, makefile and some fixes

Open svg153 opened this issue 2 years ago • 2 comments

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.

svg153 avatar Jul 22 '22 11:07 svg153

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.

alejoar avatar Jul 28 '22 08:07 alejoar

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.

svg153 avatar Aug 24 '22 11:08 svg153