Improved message when not using parametrized variable

[jjanczyszyn]

Thanks for submitting a PR, your contribution is really appreciated!

Here's a quick checklist that should be present in PRs:

• Target: for bug or doc fixes, target master; for new features, target features;
• Make sure to include one or more tests for your change;
• Add yourself to AUTHORS;
• Add a new entry to CHANGELOG.rst
  • Choose any open position to avoid merge conflicts with other PRs.
  • Add a link to the issue you are fixing (if any) using RST syntax.
  • The pytest team likes to have people to acknowledged in the CHANGELOG, so please add a thank note to yourself ("Thanks @user for the PR") and a link to your GitHub profile. It may sound weird thanking yourself, but otherwise a maintainer would have to do it manually before or after merging instead of just using GitHub's merge button. This makes it easier on the maintainers to merge PRs.

[jjanczyszyn]

@hackebrot

[hackebrot]

Waiting for CI, but code LGTM 👍

[coveralls]

Coverage increased (+0.002%) to 93.043% when pulling bbc7f3a on tramwaj29:improved-message-when-not-using-parametrized-variable into ae07985 on pytest-dev:features.

[nicoddemus]

I think the message looks good, I just think it should go under the Changes section instead of New Features.

[nicoddemus]

Thanks! Aside from my minor comments, looks good! 👍

[hackebrot]

Cool, happy to merge once @tramwaj29 addressed the comments.

[nicoddemus]

(@hackebrot if you could do it great, I will have to go out to lunch in a few minutes 😁)

[hackebrot]

No worries, I won't be until tomorrow, I think. There's no rush really 😄 We'll get it in before we release 3.0 for sure.

[jjanczyszyn]

@hackebrot I'm not sure why the CI fails, tests on my local machine passed, I tried to merge changes from features branch into mine but there were none.

[hackebrot]

@tramwaj29 there seems to be a conflict in CHANGELOG.rst. Did you update your local features branch with the upstream remote repository?

[jjanczyszyn]

@hackebrot yeah sorry forgot about that 🐶

[coveralls]

Coverage increased (+0.002%) to 93.062% when pulling 4aede6f on tramwaj29:improved-message-when-not-using-parametrized-variable into 9891593 on pytest-dev:features.

[hackebrot]

Thank you @tramwaj29! 😃