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

A hash ( # ) should not be stripped from approval name #27

Open
markijbema opened this issue Dec 18, 2013 · 3 comments
Open

A hash ( # ) should not be stripped from approval name #27

markijbema opened this issue Dec 18, 2013 · 3 comments

Comments

@markijbema
Copy link
Contributor

I noticed I test a lot of controller actions, so my names tend to be something like

"blogposts#show.json should stay the same"

I think we can do better handling of the # than stripping it. (this would mean a breaking change though, but I think this is still okay, as long as we add it to the changelog)

@kytrinyx
Copy link
Contributor

We still haven't hit 1.0, so breaking changes are considered acceptable.

How do you want to handle it. Replace it with a hyphen, perhaps?

@markijbema
Copy link
Contributor Author

Yeah, hyphen or underscore seems fine (i recall that everything is changed to underscore, but I don't have the conversion at hand).

Btw, I'm perfectly willing to implement this somewhere in the coming weeks, but I noticed it, so just jotted it down here, so I don't forget again (because I noticed it before ;) )

@kytrinyx
Copy link
Contributor

Ok, cool -- and agreed, this is a good change.

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

No branches or pull requests

2 participants