Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

travis: Add variable needed to run static checks #664

Merged

Conversation

chavafg
Copy link
Contributor

@chavafg chavafg commented Aug 29, 2018

Now that we support multiple branches, we changed how
static-checks.sh compares branches. We now need a
variable called $target_branch to make the correct
comparison when testing a PR.

Fixes: #663.

Signed-off-by: Salvador Fuentes [email protected]

Now that we support multiple branches, we changed how
static-checks.sh compares branches. We now need a
variable called $target_branch to make the correct
comparison when testing a PR.

Fixes: kata-containers#663.

Signed-off-by: Salvador Fuentes <[email protected]>
@egernst egernst added the review label Aug 29, 2018
Copy link

@devimc devimc left a comment

Choose a reason for hiding this comment

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

lgtm

@jodh-intel
Copy link
Contributor

jodh-intel commented Aug 29, 2018

lgtm

Approved with PullApprove

Copy link
Member

@amshinde amshinde left a comment

Choose a reason for hiding this comment

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

lgtm

@opendev-zuul
Copy link

opendev-zuul bot commented Aug 29, 2018

Build failed (third-party-check pipeline) integration testing with
OpenStack. For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@katacontainersbot
Copy link
Contributor

PSS Measurement:
Qemu: 169417 KB
Proxy: 4187 KB
Shim: 8890 KB

Memory inside container:
Total Memory: 2043464 KB
Free Memory: 2004008 KB

@codecov
Copy link

codecov bot commented Aug 29, 2018

Codecov Report

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

@@            Coverage Diff            @@
##             master     #664   +/-   ##
=========================================
  Coverage          ?   65.35%           
=========================================
  Files             ?       85           
  Lines             ?     9879           
  Branches          ?        0           
=========================================
  Hits              ?     6456           
  Misses            ?     2766           
  Partials          ?      657

@caoruidong caoruidong merged commit f14660f into kata-containers:master Aug 30, 2018
@egernst egernst removed the review label Aug 30, 2018
@sboeuf sboeuf added the enhancement Improvement to an existing feature label Sep 12, 2018
@chavafg chavafg deleted the topic/travis-static-checks branch February 14, 2019 20:29
egernst pushed a commit to egernst/runtime that referenced this pull request Feb 9, 2021
We should use an alias for opentracing.Span as this can be changed in the
future when we implement other tracing methods.

Fixes kata-containers#664

Signed-off-by: Gabriela Cervantes <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants