Skip to content
This repository was archived by the owner on May 4, 2019. It is now read-only.

Adapt to removal of _deleteat_(beg/end)!() and _growat_(beg/end)!() from Julia #127

Merged
merged 1 commit into from
Jul 2, 2016

Conversation

nalimilan
Copy link
Member

Update splice!() to reflect changes in the corresponding Base function.
This fixes the tests on recent Julia 0.5 master.

…rom Julia

Update splice!() to reflect changes in the corresponding Base function.
This fixes the tests on recent Julia 0.5 master.
@codecov-io
Copy link

codecov-io commented Jul 1, 2016

Current coverage is 82.03%

Merging #127 into master will decrease coverage by 1.11%

@@             master       #127   diff @@
==========================================
  Files            13         13          
  Lines           718        729    +11   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            597        598     +1   
- Misses          121        131    +10   
  Partials          0          0          

Powered by Codecov. Last updated by 61398f3...ce9eceb

@coveralls
Copy link

coveralls commented Jul 1, 2016

Coverage Status

Coverage decreased (-1.1%) to 82.03% when pulling ce9eceb on nl/deleteat into 61398f3 on master.

@nalimilan
Copy link
Member Author

@davidagold Good to merge?

@davidagold
Copy link
Contributor

Yes.

On Saturday, July 2, 2016, Milan Bouchet-Valat [email protected]
wrote:

@davidagold https://github.com/davidagold Good to merge?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#127 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ALUCm656A0l3lfmw3eFiuHkmazmY4yICks5qRjQzgaJpZM4JDMt-
.

@nalimilan nalimilan merged commit 383661a into master Jul 2, 2016
@nalimilan nalimilan deleted the nl/deleteat branch July 2, 2016 14:05
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.

4 participants