Skip to content

Add nomodule tag#2230

Merged
ChrisLovering merged 3 commits into
python-discord:mainfrom
meatballs:create-nomodule-tag
Nov 2, 2022
Merged

Add nomodule tag#2230
ChrisLovering merged 3 commits into
python-discord:mainfrom
meatballs:create-nomodule-tag

Conversation

@meatballs
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

@CaedenPH CaedenPH left a comment

Choose a reason for hiding this comment

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

Explains well in detail about the module not found and how to fix but doesn't necessarily mention about what a module actually is. Seems more like a temporary fix rather than teaching the reader what exactly a module is and how to avoid module not found.

Comment thread bot/resources/tags/nomodule.md Outdated
```
python -m pip install <your_package>
```
(You may have to specify the full path to the python executable).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is slightly ambiguous and may confuse the reader

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Agreed. It may be best to leave it out unless it can be explained briefly and clearly how "xxx -m pip install ..." means the package is available if using "xxx file.py"/"xxx -m mymodule" to run the code.

@Xithrius
Copy link
Copy Markdown
Contributor

@meatballs Hello. It seems you have a request for change on this PR.

Please resolve it, so we're able to get this merged.

Thanks!

@Xithrius Xithrius added t: feature New feature or request p: 3 - low Low Priority s: waiting for author Waiting for author to address a review or respond to a comment a: tags Related to bot tags t: enhancement Changes or improvements to existing features and removed t: feature New feature or request labels Oct 26, 2022
@meatballs
Copy link
Copy Markdown
Contributor Author

What's the requested change? I can see an opinion but it's not one I agree with.

@MarkKoz
Copy link
Copy Markdown
Contributor

MarkKoz commented Oct 27, 2022

Explains well in detail about the module not found and how to fix but doesn't necessarily mention about what a module actually is. Seems more like a temporary fix rather than teaching the reader what exactly a module is and how to avoid module not found.

Tags must be limited in scope. Otherwise they become long and loose their effectiveness. I don't think it's possible to explain what a module is in this tag in addition to what the error is without making the content long and overwhelming. Did you have something in mind that would suggest otherwise?

Comment thread bot/resources/tags/nomodule.md Outdated
@@ -0,0 +1,15 @@
**ModuleNotFoundError**

If you've installed a package but you're getting a ModuleNotFoundError, it's likely that the environment where your code is running is different from the one where you did the installation.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What do you think about briefly clarifying here that the error happens upon importing the purportedly installed package?

Copy link
Copy Markdown
Member

@ChrisLovering ChrisLovering left a comment

Choose a reason for hiding this comment

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

lgtm!

@ChrisLovering ChrisLovering merged commit 2cc592d into python-discord:main Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tags Related to bot tags p: 3 - low Low Priority s: waiting for author Waiting for author to address a review or respond to a comment t: enhancement Changes or improvements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Module Not Found

5 participants