Skip to content

fix: use full version string in _check_version for <= and < operators - #1379

Merged
bact merged 4 commits into
PyThaiNLP:devfrom
phoneee:fix/check-version-string-slice
Mar 31, 2026
Merged

fix: use full version string in _check_version for <= and < operators#1379
bact merged 4 commits into
PyThaiNLP:devfrom
phoneee:fix/check-version-string-slice

Conversation

@phoneee

@phoneee phoneee commented Mar 29, 2026

Copy link
Copy Markdown
Contributor

What do these changes do

Fix _check_version using temp[0] (first character) instead of temp (full version string) for <= and < operators

Fixes #1389

  • Passed code styles and structures
  • Passed code linting checks and unit test

@bact bact added bug bugs in the library corpus corpus/dataset-related issues labels Mar 29, 2026
@bact bact added this to PyThaiNLP Mar 29, 2026
Comment thread tests/core/test_corpus.py Outdated
@bact bact moved this to In progress in PyThaiNLP Mar 29, 2026
Comment thread tests/core/test_corpus.py Outdated
Comment thread tests/core/test_corpus.py Outdated
@sonarqubecloud

Copy link
Copy Markdown

@bact
bact merged commit 9dfae5a into PyThaiNLP:dev Mar 31, 2026
25 checks passed
@github-project-automation github-project-automation Bot moved this from In progress to Done in PyThaiNLP Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug bugs in the library corpus corpus/dataset-related issues

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

bug: _check_version returns False for all <= and < version constraints

2 participants