Skip to content
This repository was archived by the owner on Feb 12, 2022. It is now read-only.

Conversation

@dabonnie
Copy link
Contributor

@dabonnie dabonnie commented Oct 22, 2020

Consolidate workflows into a single yml file.

Signed-off-by: Devin Bonnie [email protected]

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@dabonnie dabonnie force-pushed the dabonnie/consolidate branch from 8c18d47 to f79fbeb Compare October 22, 2020 23:10
@dabonnie dabonnie changed the title Test consildation Test Consolidation Oct 22, 2020
@dabonnie dabonnie force-pushed the dabonnie/consolidate branch 5 times, most recently from 8e265f7 to 458b6e2 Compare October 22, 2020 23:45
Signed-off-by: Devin Bonnie <[email protected]>
@dabonnie dabonnie force-pushed the dabonnie/consolidate branch from 458b6e2 to 342b2a4 Compare October 22, 2020 23:50
Signed-off-by: Devin Bonnie <[email protected]>
Copy link
Contributor

@emersonknapp emersonknapp left a comment

Choose a reason for hiding this comment

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

Comments should not be considered blocking - food for thought if you want to look into them

@codecov
Copy link

codecov bot commented Oct 23, 2020

Codecov Report

Merging #65 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #65   +/-   ##
=======================================
  Coverage   89.60%   89.60%           
=======================================
  Files          18       18           
  Lines         885      885           
=======================================
  Hits          793      793           
  Misses         92       92           
Flag Coverage Δ
#ROS_1 89.60% <ø> (ø)
#ROS_2 ?
#dashing ?
#kinetic 89.69% <ø> (ø)
#melodic 91.76% <ø> (ø)

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 3d9bd8d...99e9266. Read the comment docs.

@dabonnie dabonnie marked this pull request as ready for review October 23, 2020 01:59
@dabonnie dabonnie force-pushed the dabonnie/consolidate branch from a71aac2 to 4ab0153 Compare October 23, 2020 16:54
Update build_type name

Signed-off-by: Devin Bonnie <[email protected]>
@dabonnie dabonnie force-pushed the dabonnie/consolidate branch from 4ab0153 to e1e9b35 Compare October 23, 2020 17:00
@dabonnie dabonnie force-pushed the dabonnie/consolidate branch from 77ed35c to c743f3c Compare October 23, 2020 17:36
@dabonnie dabonnie changed the title Test Consolidation Consolidation Workflows Oct 23, 2020
@dabonnie dabonnie changed the title Consolidation Workflows Consolidate Workflows Oct 23, 2020
Signed-off-by: Devin Bonnie <[email protected]>
@dabonnie dabonnie merged commit 05a071e into master Oct 23, 2020
@dabonnie dabonnie deleted the dabonnie/consolidate branch October 23, 2020 19:52
dabonnie added a commit that referenced this pull request Oct 26, 2020
dabonnie added a commit that referenced this pull request Oct 26, 2020
* Revert "Fix release latest on schedule (#66)"

This reverts commit 3811724.

* Revert "Consolidate Workflows (#65)"

This reverts commit 05a071e.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants