Skip to content

Conversation

utkarshkukreti
Copy link
Contributor

Fixes #314.

All old tests and the new test I added pass, and I also confirmed manually that the test case reported in #314 passes with this change. Let me know if you can think of any case that my change doesn't handle!

@BurntSushi
Copy link
Member

Thanks for fixing this! This was a pretty silly mistake on my part!

@BurntSushi
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jan 2, 2017

📌 Commit a9f9c9c has been approved by BurntSushi

@bors
Copy link
Contributor

bors commented Jan 2, 2017

⌛ Testing commit a9f9c9c with merge 1b2e650...

bors added a commit that referenced this pull request Jan 2, 2017
Fix replace_all when only match is at start and replacement is empty.

Fixes #314.

All old tests and the new test I added pass, and I also confirmed manually that the test case reported in #314 passes with this change. Let me know if you can think of any case that my change doesn't handle!
@bors
Copy link
Contributor

bors commented Jan 2, 2017

💔 Test failed - status-travis

@BurntSushi
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Jan 2, 2017

⌛ Testing commit a9f9c9c with merge 47661e7...

bors added a commit that referenced this pull request Jan 2, 2017
Fix replace_all when only match is at start and replacement is empty.

Fixes #314.

All old tests and the new test I added pass, and I also confirmed manually that the test case reported in #314 passes with this change. Let me know if you can think of any case that my change doesn't handle!
@bors
Copy link
Contributor

bors commented Jan 2, 2017

💔 Test failed - status-travis

@BurntSushi
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Jan 2, 2017

⌛ Testing commit a9f9c9c with merge fc639cf...

bors added a commit that referenced this pull request Jan 2, 2017
Fix replace_all when only match is at start and replacement is empty.

Fixes #314.

All old tests and the new test I added pass, and I also confirmed manually that the test case reported in #314 passes with this change. Let me know if you can think of any case that my change doesn't handle!
@bors
Copy link
Contributor

bors commented Jan 2, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: BurntSushi
Pushing fc639cf to master...

@bors bors merged commit a9f9c9c into rust-lang:master Jan 2, 2017
@mhristache
Copy link

will a new version including this fix be pushed automatically to crates.io or someone has to trigger that manually?

thanks

@BurntSushi
Copy link
Member

@maximih I'm working on it. I'll issue a release once #323 is merged.

@BurntSushi
Copy link
Member

@maximih 0.2.1 is out!

@mhristache
Copy link

@BurntSushi thanks!

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.

4 participants