Skip to content

Conversation

@JBAhire
Copy link
Member

@JBAhire JBAhire commented Dec 15, 2020

Description

While trying POC on GitHub action we realised that docker 19.03 and Java 11 are part of runner that comes with Ubuntu-20.04 already has those requirements satisfied. (ref: https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu2004-README.md)

Testing

NA

Documentation

@JBAhire JBAhire requested a review from a team December 15, 2020 10:32
@codecov
Copy link

codecov bot commented Dec 15, 2020

Codecov Report

Merging #89 (f2f4538) into main (8a8e32f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main      #89   +/-   ##
=========================================
  Coverage     68.75%   68.75%           
  Complexity      815      815           
=========================================
  Files            83       83           
  Lines          3441     3441           
  Branches        367      367           
=========================================
  Hits           2366     2366           
  Misses          931      931           
  Partials        144      144           
Flag Coverage Δ Complexity Δ
unit 68.75% <ø> (ø) 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 8a8e32f...f2f4538. Read the comment docs.

matrix:
docker-version: [19.03]
runs-on: ubuntu-latest
runs-on: ubuntu-20.04
Copy link
Contributor

Choose a reason for hiding this comment

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

can you add a comment here?
The workflow needs docker and java versions, so it's better to know against what version it has been tested.

@JBAhire JBAhire merged commit 31b027a into main Dec 17, 2020
@JBAhire JBAhire deleted the optimised-e2e branch December 17, 2020 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants