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

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

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.

@ghost ghost 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 #33 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #33   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           4      4           
  Lines         264    264           
=====================================
  Hits          264    264

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 a9a9de3...a9f1a08. Read the comment docs.

@callmehiphop callmehiphop merged commit 9ecef0d 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 19:33
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.

2 participants