Skip to content

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

Merged
bact merged 4 commits intoPyThaiNLP: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 intoPyThaiNLP:devfrom
phoneee:fix/check-version-string-slice

Conversation

@phoneee
Copy link
Copy Markdown
Contributor

@phoneee phoneee commented Mar 29, 2026

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
@bact bact moved this to In progress in PyThaiNLP Mar 29, 2026
@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