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

chore: one more workaround for repo-tools EPERM #46

Merged
merged 1 commit into from
Mar 29, 2018

Conversation

alexander-fenster
Copy link
Contributor

Sometimes it just happens, only in CircleCI and never reproduced. Here is a proof that this chmod fixes the problem: https://circleci.com/gh/googleapis/nodejs-speech/1376 - let's apply this to all our repos and see if it fails or not.

This PR fixes system tests and sample tests which were missed by previous PR.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 28, 2018
@codecov
Copy link

codecov bot commented Mar 29, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #46   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines           5      5           
=====================================
  Hits            5      5

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 28d72f3...df96a6f. Read the comment docs.

@callmehiphop callmehiphop merged commit cb647a5 into master Mar 29, 2018
@ghost ghost removed the cla: yes This human has signed the Contributor License Agreement. label Mar 29, 2018
@jmdobry jmdobry deleted the repo-tools-eperm-workaround-2 branch October 16, 2018 17:23
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