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

no-describe-selectors does not work for cucumber step_definitions #49

Open
jlin412 opened this issue Aug 12, 2016 · 7 comments
Open

no-describe-selectors does not work for cucumber step_definitions #49

jlin412 opened this issue Aug 12, 2016 · 7 comments
Milestone

Comments

@jlin412
Copy link

jlin412 commented Aug 12, 2016

Is there that we can support cucumber step_definitions as well for this rule?

@alecxe alecxe self-assigned this Aug 12, 2016
@alecxe
Copy link
Owner

alecxe commented Aug 12, 2016

@jlin412 good idea. I was afraid I should not be sticking to jasmine/mocha framework style in the plugin that much. Do you think it would be better to generalize no-describe-selectors to also support cucumber, or create a separate rule (set of rules) specifically for cucumber?

Thanks!

@igniteram
Copy link

@alecxe I would suggest to have to separate set of rules as there many contrasting features between jasmine and cucumber . Let me see if could send some PR's!

@alecxe alecxe removed their assignment Aug 12, 2016
@alecxe
Copy link
Owner

alecxe commented Aug 12, 2016

@igniteram would really appreciate it. Let me know if you need any help or any adjustments to the current rules or exposed settings to support the new cucumber rules. Thank you.

@alecxe
Copy link
Owner

alecxe commented Aug 12, 2016

@igniteram by the way, you can ping me on gitter whenever needed

@igniteram
Copy link

Sure @alecxe will do that 👍

@jlin412
Copy link
Author

jlin412 commented Aug 13, 2016

Just throw it out there that no-get-in-it
https://github.com/alecxe/eslint-plugin-protractor/blob/master/docs/rules/no-get-in-it.md
rule
doesn't apply in cucumber step definition though it is not as critical as
the other one.
I am happy to help test if there is a PR.

On Fri, Aug 12, 2016 at 3:39 PM, Ram Pasala notifications@github.com
wrote:

Sure @alecxe https://github.com/alecxe will do that 👍


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

@alecxe
Copy link
Owner

alecxe commented Aug 13, 2016

@jlin412 gotcha, please disable the rules that don't apply to your particular case.

@alecxe alecxe added this to the 2.0 milestone Jun 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants