Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unit test workin in Py2 and Py3 #88

Merged
merged 16 commits into from
Feb 15, 2022
Merged

Unit test workin in Py2 and Py3 #88

merged 16 commits into from
Feb 15, 2022

Conversation

vokimon
Copy link
Member

@vokimon vokimon commented Feb 9, 2022

The following changes are done:

  • Add github actions workflows for CI that install dependencies in Py2 and Py3 and pass the unittests. Functional tests (using erppeek) and Destral tests are not run yet.
  • Add a gha workflow to publish the non-erp module as we push the version tag.
  • Dependencies somutils and plantmonitor have been put under similar workflows (CI and pypi publish).
  • somutils and plantmonitor are already published. generationkwh is not, it should be published after this branch is merged.
  • To get dependencies (somutils and plantmonitor) in green, conditional dependencies for Py2.7 had to be set as several libraries stopped suporting Py2. Althought it works standalone, interactions with dependencies of a full erp installation is to be checked. Conditional restrictions are documented in setup.py of each project.
  • Commonalities (isodate, run_destructive_test and so on...) were moved to somutils.
  • Removed wavefile dependency (from b2btest, now modularized)
  • success and coverage badges are included both in the README and in qa-control, besides the circle-ci badges

@vokimon
Copy link
Member Author

vokimon commented Feb 9, 2022

TODO: El coverage se entrelaza con el de Cercle-ci

@oriolpiera oriolpiera merged commit aa3f0e3 into master Feb 15, 2022
@oriolpiera oriolpiera deleted the Py2-somutils branch February 15, 2022 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants