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

fix cleanpull() #181

Merged
merged 1 commit into from
Jun 18, 2019
Merged

fix cleanpull() #181

merged 1 commit into from
Jun 18, 2019

Conversation

cserteGT3
Copy link
Contributor

In short:
old:

julia>  print(rpad("→ Retrieving updates from the repository..."), 35)
ERROR: MethodError: no method matching rpad(::String)
Closest candidates are:
  rpad(::Union{AbstractChar, AbstractString}, ::Integer) at strings/util.jl:269
  rpad(::Union{AbstractChar, AbstractString}, ::Integer, ::Union{AbstractChar, AbstractString}) at strings/util.jl:269
  rpad(::Any, ::Integer) at strings/util.jl:262
  ...
Stacktrace:
 [1] top-level scope at none:0

Fix:

julia>  print(rpad("→ Retrieving updates from the repository...", 35))
→ Retrieving updates from the repository...

@codecov
Copy link

codecov bot commented Jun 17, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #181   +/-   ##
=======================================
  Coverage   88.18%   88.18%           
=======================================
  Files          25       25           
  Lines        1007     1007           
=======================================
  Hits          888      888           
  Misses        119      119
Impacted Files Coverage Δ
src/manager/post_processing.jl 28.57% <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 1eb6987...98ae1ab. Read the comment docs.

@tlienart
Copy link
Owner

Great, thanks!

@tlienart tlienart merged commit 8027a42 into tlienart:master Jun 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants