Skip to content

Conversation

@bgn42
Copy link
Collaborator

@bgn42 bgn42 commented Feb 12, 2025

Fixes

#386

Changes

update not only index.pidx but also old pdsc in .web
fixed missing check for range comparison => error if not in range
display range 1.2.3:_ as >=1.2.3 in messages

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 81f3b8b and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 46.1% (50% is the threshold).

This pull request will bring the total coverage in the repository to 58.7% (0.0% change).

View more on Code Climate.

@bgn42 bgn42 requested a review from jkrech February 12, 2025 15:56
Copy link
Member

@jkrech jkrech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bgn42 bgn42 merged commit 9d6383c into main Feb 12, 2025
19 of 20 checks passed
@bgn42 bgn42 deleted the notInstalled branch February 12, 2025 16:05
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.

3 participants