-
Notifications
You must be signed in to change notification settings - Fork 119
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
Add pre-release test to CI [melodic-devel] #69
Conversation
6d6bfdb
to
cbb5285
Compare
662d5fb
to
0514c50
Compare
Doing prerelease tests automatically for PRs would be a lot of overhead I don't like.
|
0514c50
to
0f0fa92
Compare
I updated this so it works just like the one on master now. The pre-release test for this repo only takes ~10min and it is nice being able to see if it is passing when going to the repo. |
c182966
to
8d0ad09
Compare
Per @rhaschke comment in the master branch version of this PR I also updated this to test moveit as a downstream dependency as part of the pre-release test. |
8d0ad09
to
43a6d38
Compare
Pushed the same changes here as I made on the PR for master. |
92fe636
to
03aae33
Compare
Let's finish master first (#70) and then apply the same changes here. |
03aae33
to
c4a84c5
Compare
Closing since this is merged with #83. |
This adds the pre-release test to the melodic-devel branch. Since this is a release branch I believe adding it to all PRs made against this branch will be useful.