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

replace Array{...}(shape...)-like calls in test/r* #24752

Merged
merged 1 commit into from
Nov 24, 2017

Conversation

Sacha0
Copy link
Member

@Sacha0 Sacha0 commented Nov 24, 2017

This pull request replaces Array{...}(shape...)-like calls in test/r*.jl. Ref. #24595. Best!

@Sacha0 Sacha0 added the arrays [a, r, r, a, y, s] label Nov 24, 2017
@fredrikekre
Copy link
Member

Error in testset libgit2:
Error During Test at nothing:1
  Test threw an exception of type RemoteException
  Expression: libgit2
  Error deserializing a remote exception from worker 7
  Remote(original) exception of type Test.TestSetException
  Remote stacktrace : 
  finish at /home/circleci/project/usr/share/julia/site/v0.7/Test/src/Test.jl:758
  macro expansion at ./util.jl:310 [inlined]
  top-level scope at ./<missing>:19
  eval at ./boot.jl:284
  #runtests#1 at /tmp/julia/share/julia/test/testdefs.jl:25
  #runtests at ./<missing>:0 (repeats 2 times)
  #114 at ./distributed/process_messages.jl:271
  run_work_thunk at ./distributed/process_messages.jl:56
  macro expansion at ./distributed/process_messages.jl:271 [inlined]
  #113 at ./event.jl:95
ERROR: LoadError: Test run finished with errors
in expression starting at /tmp/julia/share/julia/test/runtests.jl:44

on circle. Relevant test passed.

@fredrikekre fredrikekre merged commit 4eb5cde into JuliaLang:master Nov 24, 2017
@Sacha0 Sacha0 deleted the reparrtr branch November 24, 2017 20:30
@Sacha0
Copy link
Member Author

Sacha0 commented Nov 24, 2017

Thanks @fredrikekre! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arrays [a, r, r, a, y, s]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants