venkateshwaracholan opened a new pull request, #4819:
URL: https://github.com/apache/polaris/pull/4819
## Summary
The Python CLI was printing a profile loading message every time profile
data was read from disk:
```text
Loading profiles from ...
```
This message appeared during normal profile operations such as `polaris
profiles list` and `polaris profiles get <profile>`. While useful during
development, it does not provide actionable information for most users and adds
unnecessary noise to command output.
The extra output can also be problematic for scripts and automation that
expect profile commands to return only the requested data. This change removes
the message to keep CLI output clean and focused on user-facing information.
## Changes
* Removed the unconditional profile loading message from `_load_profiles()`.
* Added regression tests to ensure:
* `profiles list` does not print the loading message.
* `profiles get` does not print the loading message.
* Verified that the existing command output and behavior remain unchanged
apart from the removal of the extra message.
## Checklist
* [ ] ๐ก๏ธ Don't disclose security issues! (contact
[[[email protected]](mailto:[email protected])](mailto:[email protected]))
* [ ] ๐ Clearly explained why the changes are needed, or linked related
issues: Fixes #4818
* [ ] ๐งช Added/updated tests with good coverage, or manually tested (and
explained how)
* [ ] ๐ก Added comments for complex logic
* [ ] ๐งพ Updated `CHANGELOG.md` (if needed)
* [ ] ๐ Updated documentation in `site/content/in-dev/unreleased` (if needed)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
