Skip to content
This repository has been archived by the owner on Jul 20, 2023. It is now read-only.

run circleci as non-root user #24

Merged
merged 1 commit into from
Feb 16, 2018
Merged

Conversation

alexander-fenster
Copy link
Contributor

It saves several minutes per CI task because it keeps recompiling grpc being unable to unpack it. Some details here.

  • Tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

It saves several minutes per CI task because it keeps recompiling `grpc` being unable to unpack it. Some details [here](ForbesLindesay-Unmaintained/tar-pack#35).
@codecov-io
Copy link

codecov-io commented Feb 14, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #24   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines           4      4           
=====================================
  Hits            4      4

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 f2b5c09...bbf7b00. Read the comment docs.

@alexander-fenster
Copy link
Contributor Author

@lukesneeringer or @matthewayne - ping please :) This one will speed up CI a lot (we don't have any better solution so far).

@alexander-fenster alexander-fenster merged commit a7222be into master Feb 16, 2018
@alexander-fenster alexander-fenster deleted the run-circle-nonroot branch February 16, 2018 18:20
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.

3 participants