Skip to content

Revert "Revert "Remove chardet version upper limit""#7263

Merged
yarikoptic merged 2 commits into
datalad:maintfrom
yarikoptic:rerevert-chardet
Jan 18, 2023
Merged

Revert "Revert "Remove chardet version upper limit""#7263
yarikoptic merged 2 commits into
datalad:maintfrom
yarikoptic:rerevert-chardet

Conversation

@yarikoptic

Copy link
Copy Markdown
Member

This reverts commit 09cb41c. Original PR which removed upper bound:
#7029

Debians now have 5.1.0, and we can't package while demanding an outdated chardet. Let's see again if we really need to limit upper version.

This reverts commit 09cb41c.
Original PR which removed upper bound:
datalad#7029

Debians now have 5.1.0, and we can't package while demanding an outdated
chardet. Let's see again if we really need to limit upper version.
@yarikoptic yarikoptic added semver-dependencies Update one or more dependencies version CHANGELOG-missing When a PR's description does not contain a changelog item, yet. labels Jan 17, 2023
@github-actions github-actions Bot removed the CHANGELOG-missing When a PR's description does not contain a changelog item, yet. label Jan 17, 2023
@codecov

codecov Bot commented Jan 17, 2023

Copy link
Copy Markdown

Codecov Report

Base: 88.72% // Head: 90.70% // Increases project coverage by +1.98% 🎉

Coverage data is based on head (dfae681) compared to base (c92ae3a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##            maint    #7263      +/-   ##
==========================================
+ Coverage   88.72%   90.70%   +1.98%     
==========================================
  Files         326      326              
  Lines       44423    44423              
  Branches     5919     5919              
==========================================
+ Hits        39415    40296     +881     
+ Misses       4993     4112     -881     
  Partials       15       15              
Impacted Files Coverage Δ
datalad/customremotes/ria_remote.py 55.55% <0.00%> (-11.12%) ⬇️
datalad/_version.py 45.68% <0.00%> (ø)
datalad/support/tests/test_annexrepo.py 98.03% <0.00%> (+0.13%) ⬆️
datalad/__init__.py 98.00% <0.00%> (+16.00%) ⬆️
datalad/tests/utils.py 52.75% <0.00%> (+52.75%) ⬆️
datalad/tests/test_tests_utils.py 92.89% <0.00%> (+92.89%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@yarikoptic

Copy link
Copy Markdown
Member Author

all tests pass, debian build also passed. Let's merge. Third time is a charm, right?

@yarikoptic
yarikoptic merged commit a9eb64a into datalad:maint Jan 18, 2023
@yarikoptic-gitmate

Copy link
Copy Markdown
Collaborator

PR released in 0.18.2

@yarikoptic
yarikoptic deleted the rerevert-chardet branch February 13, 2025 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver-dependencies Update one or more dependencies version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants