Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copy using io.Copy instead of os.Rename #732

Merged
merged 6 commits into from
Mar 3, 2020
Merged

Conversation

rberrelleza
Copy link
Member

Fixes #731

@derek
Copy link

derek bot commented Mar 3, 2020

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide.
Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

1 similar comment
@derek
Copy link

derek bot commented Mar 3, 2020

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide.
Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

@derek derek bot added the no-dco label Mar 3, 2020
@derek
Copy link

derek bot commented Mar 3, 2020

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide.
Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

@codecov
Copy link

codecov bot commented Mar 3, 2020

Codecov Report

Merging #732 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #732   +/-   ##
=======================================
  Coverage   31.98%   31.98%           
=======================================
  Files          59       59           
  Lines        4627     4627           
=======================================
  Hits         1480     1480           
  Misses       3048     3048           
  Partials       99       99
Impacted Files Coverage Δ
pkg/okteto/auth.go 17.67% <0%> (ø) ⬆️

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 6f9212a...7b75427. Read the comment docs.

Signed-off-by: Ramiro Berrelleza <rberrelleza@gmail.com>
t.Errorf("fail to detect non-existing file")
}

if err := ioutil.WriteFile(p, []byte("hello-world"), 600); err != nil {

Choose a reason for hiding this comment

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

SA9002: file mode '600' evaluates to 01130; did you mean '0600'? (from staticcheck)

Signed-off-by: Ramiro Berrelleza <rberrelleza@gmail.com>
t.Errorf("fail to detect non-existing file")
}

if err := ioutil.WriteFile(p, []byte("hello-world"), 600); err != nil {

Choose a reason for hiding this comment

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

SA9002: file mode '600' evaluates to 01130; did you mean '0600'? (from staticcheck)

@rberrelleza rberrelleza removed the no-dco label Mar 3, 2020
@derek derek bot added the no-dco label Mar 3, 2020
@derek
Copy link

derek bot commented Mar 3, 2020

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide.
Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

@rberrelleza rberrelleza merged commit ee211d3 into master Mar 3, 2020
@rberrelleza rberrelleza deleted the copy-using-shutil branch March 3, 2020 03:27
@feitian124
Copy link

thanks , i happen to met this issue , after upgrade it is fine now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

download syncthing fails if /tmp is in a different partition
4 participants