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

[enhancement] Prepare travis-ci build #68

Closed
2 of 3 tasks
nepython opened this issue May 5, 2020 · 2 comments · May be fixed by #164
Closed
2 of 3 tasks

[enhancement] Prepare travis-ci build #68

nepython opened this issue May 5, 2020 · 2 comments · May be fixed by #164
Assignees
Labels
timeseries Issues / PRs / tasks related to timeseries database

Comments

@nepython
Copy link
Contributor

nepython commented May 5, 2020

Prepare a travis-ci build which runs tests with both timeseries DB supported, performs QA checks (like other equivalent openwisp modules) and collects test coverage data.

The issue can be broken down too:

  • Prepare a basic travis-ci build that collects test coverage data(It has already been done thanks to @pandafy)
  • Travis also stands updated with qa checks and includes black formatting (thanks to @PabloCastellano and @nemesisdesign)
  • Prepare travis to run tests for both Timeseries DB

To be done after #66

@nepython nepython added the enhancement New feature or request label May 5, 2020
@nepython nepython self-assigned this May 5, 2020
@nepython nepython added timeseries Issues / PRs / tasks related to timeseries database and removed enhancement New feature or request labels Jul 2, 2020
@nepython nepython linked a pull request Jul 27, 2020 that will close this issue
3 tasks
@nemesifier
Copy link
Member

@nepython what's the status of this?

@nepython
Copy link
Contributor Author

@nepython what's the status of this?

It has been mostly completed in other PRs, the last part of adding support for elasticsearch so we can run tests with both the Timeseries databases together has been implemented in #164 (still open but this part of Travis is completed in it).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
timeseries Issues / PRs / tasks related to timeseries database
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants