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

Add nightly CI #53

Merged
merged 17 commits into from
Aug 19, 2021
Merged

Add nightly CI #53

merged 17 commits into from
Aug 19, 2021

Conversation

ddengster
Copy link
Contributor

@ddengster ddengster commented May 11, 2021 •
edited
Loading

Signed-off-by: ddengster ed.fan@osrfoundation.org

@ddengster ddengster marked this pull request as ready for review May 11, 2021 07:58
@ddengster ddengster mentioned this pull request May 11, 2021
Closed
@ddengster ddengster force-pushed the feature/nightlyci branch from 23789b5 to f9ba958 Compare May 11, 2021 09:03
@cnboonhan cnboonhan mentioned this pull request May 24, 2021
Open
@ddengster ddengster closed this Jul 15, 2021
@ddengster ddengster reopened this Jul 15, 2021
ddengster added 15 commits August 12, 2021 10:08
@ddengster
add nightlyci
4c976a2 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
add nightly badge
9babce5 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
formatting
b8d23af 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
add logs folder artifact upload, test run
7773001 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
syntax fix
526ceb1 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
fix cron, fix syntax
2d3998b 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
format
f5691d9 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
wip
6c0f84f 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
fix syntax
d810975 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
add PR trigger
7142cdf 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
update setup-ros to 0.2
870ec06 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
remove test_maps package
a1a979e 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
re-add test_maps, add dependencies for traffic editor
7798a65 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
export qt define
88c27ff 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
display set to 0
44fb8e9 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@ddengster
set env variable
7116442 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
@codecov
Copy link

codecov bot commented Aug 12, 2021 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (main@a7aaea0). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #53   +/-   ##
=======================================
  Coverage        ?   21.15%           
=======================================
  Files           ?      569           
  Lines           ?    51562           
  Branches        ?    24978           
=======================================
  Hits            ?    10910           
  Misses          ?    26235           
  Partials        ?    14417
Flag Coverage Δ
tests 21.15% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a7aaea0...e69f83e. Read the comment docs.

@ddengster
remove unneeded
e69f83e 
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
aaronchongth
aaronchongth approved these changes Aug 19, 2021
View reviewed changes
Copy link
Member

@aaronchongth aaronchongth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, thanks! Just to be sure, we don't intend to include message building, is that correct? i.e. rmf_internal_msgs and rmf_visualization_msgs

@ddengster
Copy link
Contributor Author

This looks great, thanks! Just to be sure, we don't intend to include message building, is that correct? i.e. rmf_internal_msgs and rmf_visualization_msgs

These are dependencies so they will be built as part of the process. If anything fails at that level we will see it.

@ddengster ddengster merged commit ab16967 into main Aug 19, 2021
@ddengster ddengster deleted the feature/nightlyci branch August 19, 2021 03:21
thedevmanek pushed a commit to thedevmanek/rmf that referenced this pull request Sep 5, 2022
@ddengster
Add nightly CI (open-rmf#53)
2c0e826 
* add nightlyci

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* add nightly badge

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* formatting

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* add logs folder artifact upload, test run

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* syntax fix

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* fix cron, fix syntax

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* format

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* wip

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* fix syntax

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* add PR trigger

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* update setup-ros to 0.2

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* remove test_maps package

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* re-add test_maps, add dependencies for traffic editor

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* export qt define

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* display set to 0

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* set env variable

Signed-off-by: ddengster <ed.fan@osrfoundation.org>

* remove unneeded

Signed-off-by: ddengster <ed.fan@osrfoundation.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aaronchongth aaronchongth aaronchongth approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ddengster @aaronchongth