-
Notifications
You must be signed in to change notification settings - Fork 664
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
Use correct name for Beautiful Soup in dependencies #1296
Conversation
As noted at https://pypi.org/project/bs4/, the official name of the Beautiful Soup package is beautifulsoup4, while bs4 is just an alias. In distribution packaging (such as Fedora) the package may not be available under this alias, so it's better to use the official name. Signed-off-by: Ondrej Mosnáček <omosnacek@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #1296 +/- ##
=======================================
Coverage 34.36% 34.36%
=======================================
Files 45 45
Lines 8241 8241
Branches 1146 1146
=======================================
Hits 2832 2832
Misses 5295 5295
Partials 114 114 ☔ View full report in Codecov by Sentry. |
Thanks, I have not known |
This should've been added to |
|
The official package is "beautifulsoup4", yes, but the module inside is bs4.
does not work. It should be:
See the official documentation: |
Exactly, that's the issue. The import statement should be |
This is the new error message with 3.41.6:
|
same here
|
not difficult, just follow the doc |
This is not an useful comment. The problem still arises. |
Check this PR addressing |
As noted at https://pypi.org/project/bs4/, the official name of the Beautiful Soup package is beautifulsoup4, while bs4 is just an alias. In distribution packaging (such as Fedora) the package may not be available under this alias, so it's better to use the official name.