Skip to content

fix: explain empty location loads - #233

Merged
krowvin merged 1 commit into
mainfrom
fix/load-location-empty-message-186
Jul 11, 2026
Merged

fix: explain empty location loads#233
krowvin merged 1 commit into
mainfrom
fix/load-location-empty-message-186

Conversation

@krowvin

@krowvin krowvin commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • return a friendly message when a load location source query returns no locations
  • point users toward refining --like / --location-kind-like and checking filters in CDA Swagger or the regex guide
  • avoid initializing the target session or reporting a misleading success path when there is nothing to copy

Resolves #186

Validation

  • poetry run pytest tests/commands/test_load_location_ids.py -q
  • poetry run pytest -q
  • poetry check

@krowvin
krowvin marked this pull request as ready for review July 11, 2026 20:33
@krowvin
krowvin requested a review from Enovotny as a code owner July 11, 2026 20:33
@krowvin
krowvin merged commit a5b596c into main Jul 11, 2026
9 checks passed
@krowvin
krowvin deleted the fix/load-location-empty-message-186 branch July 11, 2026 20:55
msweier pushed a commit that referenced this pull request Aug 10, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.8.0](v0.7.4...v0.8.0)
(2026-08-10)


### Features

* Add CDA Environment Manager
([#209](#209))
([a89bfb1](a89bfb1))
* add dev helper script to preview docs
([#253](#253))
([749fedc](749fedc))
* Add modern DSS transfer utilities
([#242](#242))
([89e01f4](89e01f4))
* allow for line return tsid parsing
([#230](#230))
([39f7f6b](39f7f6b))
* display CDA server stack traces in debug mode
([#247](#247))
([c3bfcf9](c3bfcf9))


### Bug Fixes

* align formatting checks
([#229](#229))
([a6286b3](a6286b3))
* cover explicit blob media type
([#232](#232))
([1a3b26d](1a3b26d))
* doc check, add link exception
([#255](#255))
([0101968](0101968))
* explain empty location loads
([#233](#233))
([a5b596c](a5b596c))
* harden clob download error handling
([#235](#235))
([621f913](621f913))
* Improve version issue error message in deps.py
([#245](#245))
([f77b75f](f77b75f))
* refresh ownership metadata
([#251](#251))
([3b31479](3b31479))
* show update environment before confirmation
([#246](#246))
([500435c](500435c)),
closes
[#222](#222)
* validate blob list limits
([#231](#231))
([0917ce9](0917ce9))
* validate load target cda
([#236](#236))
([ff3e06a](ff3e06a))
* wait for auth callback server readiness
([#249](#249))
([ca39d5b](ca39d5b))


### Documentation

* guide agents to shared color helpers
([#248](#248))
([ef03276](ef03276))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

Load locations does not return message if no locations return

1 participant