FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Unpin nltk by rosbo · Pull Request #1530 · Kaggle/docker-python · GitHub

Unpin nltk - #1530

Merged
rosbo merged 1 commit into
mainfrom
rm-pin-nltk
Jan 30, 2026
Merged

rosbo merged 1 commit into
mainfrom
rm-pin-nltk

Conversation

rosbo commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

Also removed preprocessing python package because it pins nltk: https://github.com/SpotlightData/preprocessing/blob/180c6472bc2642afbd7a1ece08d0b0d14968a708/requirements.txt#L1

The preprocessing package is unmaintained, last release was in 2017.

http://b/408298750

Also removed `preprocessing` python package because it pins `nltk`: https://github.com/SpotlightData/preprocessing/blob/180c6472bc2642afbd7a1ece08d0b0d14968a708/requirements.txt#L1

The `preprocessing` package is unmaintained, last release was in 2017.

http://b/408298750
rosbo requested a review from develra January 29, 2026 23:59

rosbo commented Jan 30, 2026

Copy link
Copy Markdown
Contributor Author

Adding you @develra given you did something similar as part of #1523.

rosbo merged commit 8a95ad1 into main Jan 30, 2026
4 checks passed
rosbo deleted the rm-pin-nltk branch January 30, 2026 00:22
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL