Conversation
There was a problem hiding this comment.
Pull request overview
This pull request prepares version 0.12.3 of BEDbase, fixing a reference genome validation bug by upgrading the bedboss dependency from version 0.8.5 to 0.8.6. The PR also includes unreported UI improvements for better user communication during service outages.
Changes:
- Updated bedboss dependency to version 0.8.6 to fix reference genome validation bug
- Added a connection banner UI component to display maintenance messages when the service is unavailable
- Updated version numbers across the codebase
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| bedhost/_version.py | Updated backend version to 0.12.3 |
| ui/package.json | Updated UI version to 0.12.2 |
| requirements/requirements-all.txt | Upgraded bedboss dependency to >=0.8.6 |
| ui/src/components/common/connection-banner.tsx | Added new ConnectionBanner component to display maintenance messages |
| ui/src/components/layout.tsx | Integrated ConnectionBanner component into both layout variations |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "name": "bedhost-ui-2", | ||
| "private": true, | ||
| "version": "0.12.0", | ||
| "version": "0.12.2", |
There was a problem hiding this comment.
The UI package version is updated to 0.12.2, but the PR title and the backend version indicate this is release 0.12.3. This version mismatch creates confusion about what version is being released. The UI version should be updated to 0.12.3 to match the release version, or the versioning strategy should be clarified.
| "version": "0.12.2", | |
| "version": "0.12.3", |
| cachetools>=4.2.4 | ||
| bedboss>=0.8.5 | ||
| bedboss>=0.8.6 | ||
| #bedboss @ git+https://github.com/databio/bedboss.git@dev#egg=bedboss |
There was a problem hiding this comment.
This commented-out line should either be removed or uncommented with a clear explanation of why it exists. Leaving commented-out code in requirements files can cause confusion about which dependencies are actually being used, especially since line 15 already specifies the required bedboss version.
| #bedboss @ git+https://github.com/databio/bedboss.git@dev#egg=bedboss |
Changes:
TODO:
__version__.pyfile