'/' is also a valid character in a MiniTest test name #212

Merged
merged 2 commits into from Nov 9, 2013

Conversation

Projects
None yet
2 participants
Contributor

modosc commented Nov 1, 2013

'/' is also a valid character in a MiniTest test name

gs commented on d444678 Nov 3, 2013

I would prefer to escape the '/' sign...because I check that in rubular

not escaped
http://cl.ly/image/1c3w3t302i05

escaped
http://cl.ly/image/0k0m2p3Q2W1c

Owner

modosc replied Nov 7, 2013

no problem, fixed.

gs replied Nov 8, 2013

did you create new request? as I dont see changes

Contributor

modosc commented Nov 8, 2013

the second commit is showing in this pull request for me, do you not see it?

gs added a commit that referenced this pull request Nov 9, 2013

Merge pull request #212 from modosc/master
'/' is also a valid character in a MiniTest test name

@gs gs merged commit bcd9ab9 into maltize:master Nov 9, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment