Skip to content

Rr 77 version print#79

Merged
ryanraaschCDC merged 8 commits into
mainfrom
rr-77-version-print
Apr 29, 2026
Merged

Rr 77 version print#79
ryanraaschCDC merged 8 commits into
mainfrom
rr-77-version-print

Conversation

@ryanraaschCDC

Copy link
Copy Markdown
Collaborator

No description provided.

@ryanraaschCDC ryanraaschCDC self-assigned this Apr 27, 2026
@ryanraaschCDC ryanraaschCDC linked an issue Apr 27, 2026 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR bumps the package version and updates the changelog to document a fix for duplicate version printing during some get_dataframe() calls, implemented by adding a print_version toggle to blob reads.

Changes:

  • Bump project version to 2026.04.27.0.
  • Add changelog entry for the duplicate version-print fix.
  • Add print_version parameter to BlobEndpoint.read_blobs() and suppress version printing from the internal read_blobs() call inside get_dataframe().

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
pyproject.toml Updates CalVer version to 2026.04.27.0.
changelog.md Documents the duplicate version-print fix in a new release entry.
cfa/dataops/catalog.py Adds print_version plumbing to avoid double-printing the chosen version during get_dataframe().

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread changelog.md Outdated
Comment thread cfa/dataops/catalog.py
Comment thread cfa/dataops/catalog.py Outdated
ryanraaschCDC and others added 4 commits April 27, 2026 10:36
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

@ksinger-cdc ksinger-cdc left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I see you're just adding in the print version arg and updating the code to reflect that. The copilot comment doesn't seem to be related to changes you've made. So long as the list comprehension doesn't break anything, should be good to go - sounds like you've tested that. LGTM.

@ryanraaschCDC
ryanraaschCDC merged commit 7be1f91 into main Apr 29, 2026
4 checks passed
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.

version printed twice in get_dataframe() call

4 participants