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

Add example for bad-super-call #6166

Merged
merged 3 commits into from
Apr 3, 2022

Conversation

DudeNr33
Copy link
Collaborator

@DudeNr33 DudeNr33 commented Apr 3, 2022

Co-authored by: Vladyslav Krylasov vladyslav.krylasov@gmail.com

  • Write a good description on what the PR does.
  • If you used multiple emails or multiple names when contributing, add your mails
    and preferred name in script/.contributors_aliases.json

Type of Changes

Type
🐛 Bug fix
✨ New feature
🔨 Refactoring
📜 Docs

Description

Ref #5953

Co-authored-by: Jacob Walls <jacobtylerwalls@gmail.com>
Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm trying to avoid foo/bar in examples because it's not friendly for beginner, (the persons that are the most likely to read the documentation). #6115 (comment)

doc/data/messages/b/bad-super-call/bad.py Outdated Show resolved Hide resolved
doc/data/messages/b/bad-super-call/details.rst Outdated Show resolved Hide resolved
doc/data/messages/b/bad-super-call/good.py Outdated Show resolved Hide resolved
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
@DudeNr33
Copy link
Collaborator Author

DudeNr33 commented Apr 3, 2022

That makes sense that this whole foobar thing might be confusing for beginners. Thanks for the review!

@Pierre-Sassoulas Pierre-Sassoulas merged commit b82183b into pylint-dev:main Apr 3, 2022
@DudeNr33 DudeNr33 deleted the doc-bad-super-call branch April 5, 2022 15:25
@Pierre-Sassoulas Pierre-Sassoulas removed this from the 2.14.0 milestone May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants