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.

Sign up for GitHub

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

Jump to bottom

Standardize CI infrastructure #28

Closed
5 tasks done
phil-davis opened this issue Nov 27, 2018 · 6 comments
Closed
5 tasks done

Standardize CI infrastructure #28

phil-davis opened this issue Nov 27, 2018 · 6 comments
Assignees
@phil-davis
Labels
2 - developing Drone QA:team

Comments

@phil-davis
Copy link
Contributor

phil-davis commented Nov 27, 2018 •
edited
Loading

  • move from Travis to drone
  • implement owncloud/coding-standard in vendor-bin
  • add test-php-unit to Makefile and use it from drone
  • move unit tests into tests/unit folder
  • review the CI infrastructure and standardize anything else

Part of issue owncloud/QA#582
@phil-davis
Copy link
Contributor Author

Also see #26 - add PHP7.2 to stable10
Include a suitable range of PHP versions and core master+stable10 in the test matrix.

@phil-davis phil-davis mentioned this issue Dec 6, 2018
Closed
@skshetry skshetry self-assigned this Dec 19, 2018
@individual-it individual-it assigned phil-davis and unassigned skshetry Feb 12, 2019
@phil-davis phil-davis mentioned this issue Feb 12, 2019
Merged
@phil-davis phil-davis mentioned this issue Mar 6, 2019
Closed
This was referenced Mar 7, 2019
Merged
Merged
Merged
Merged
@phil-davis
Copy link
Contributor Author

PRs #36 #37 #38 done - they get CI running and passing in both travis and drone

@phil-davis
Copy link
Contributor Author

phil-davis commented Mar 7, 2019 •
edited
Loading

Waiting for repo settings to be adjusted:

  • require drone
  • do not require travis
  • add codecov_token

then #39 should pass, and a new PR can be raised to delete .travis.yml and adjust the badges in README.md

@PVince81
Copy link
Contributor

PVince81 commented Mar 7, 2019

@phil-davis I've set drone/pr and removed travis for this repo.

not sure about the codecov_token, is it the same for all our repos ?

@phil-davis phil-davis mentioned this issue Mar 7, 2019
Merged
@phil-davis
Copy link
Contributor Author

@PVince81 I have no idea about codecov_token. @DeepDiver1975 has done it in other repos.

@phil-davis
Copy link
Contributor Author

codecov_token done. PR #40 merged. All actions complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@phil-davis phil-davis

Labels
2 - developing Drone QA:team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@PVince81 @phil-davis @skshetry