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.

Sign up for GitHub

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

Jump to bottom

Case-insensitive target names #230

Closed
Mpdreamz opened this issue Mar 28, 2019 · 4 comments · Fixed by #257
Closed

Case-insensitive target names #230

Mpdreamz opened this issue Mar 28, 2019 · 4 comments · Fixed by #257
Labels
breaking This change could break current consumers enhancement New feature or request
Milestone
3.0.0

Comments

@Mpdreamz
Copy link

Right now target is case sensitive, it would be nice if this was case insensitive allowing you to use nameof(Method) but pass method on the command line
@adamralph
Copy link
Owner

Hey @Mpdreamz thanks for raising this. I'm on the road right now but I'll look into this ASAP.

@adamralph adamralph changed the title Should target be case sensitive? Make target names case insensitive Mar 31, 2019
@adamralph adamralph added enhancement New feature or request breaking This change could break current consumers on-hold This can't be done yet labels Mar 31, 2019
@adamralph
Copy link
Owner

@Mpdreamz I've converted this to a breaking feature, since I can't think of any good reason not to do it, but it will have to wait for a major release. That said, there are a number of breaking features in the backlog now, so perhaps it's time to do it.

@adamralph adamralph added this to the 3.0.0 milestone Apr 6, 2019
@adamralph adamralph removed on-hold This can't be done yet labels Jul 15, 2019
@adamralph adamralph mentioned this issue Jul 16, 2019
Merged
@adamralph adamralph changed the title Make target names case insensitive Case insensitive target names Jul 24, 2019
@adamralph adamralph changed the title Case insensitive target names Case-insensitive target names Jul 24, 2019
@ghost ghost mentioned this issue Aug 21, 2019
Closed
@adamralph
Copy link
Owner

Released in 3.0.0-alpha.1.

@Mpdreamz @ygel perhaps you'd like to try it out?

@ghost
Copy link

ghost commented Aug 24, 2019 via email •
edited by adamralph
Loading

@adamralph adamralph mentioned this issue Sep 16, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking This change could break current consumers enhancement New feature or request
Projects
None yet
Milestone
3.0.0
Development

Successfully merging a pull request may close this issue.

case insensitivty adamralph/bullseye
2 participants
@Mpdreamz @adamralph