Skip to content

Fix min_version_guard #81

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
olivhoenen merged 2 commits into from
Nov 4, 2025
Merged

Fix min_version_guard #81

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
27450e2
Fix min_version_guard when AL-Core is not present
olivhoenen Nov 3, 2025
fb0b015
Remove guard as IMAS-Python dropped the AL4 support
olivhoenen Nov 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion imas/command/cli.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,6 @@ def print_ids(uri, ids, occurrence, print_all):
ids Name of the IDS to print (e.g. "core_profiles").
occurrence Which occurrence to print (defaults to 0).
"""
min_version_guard(Version("5.0"))
setup_rich_log_handler(False)

with DBEntry(uri, "r") as dbentry:
Expand Down
2 changes: 1 addition & 1 deletion imas/command/helpers.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ def min_version_guard(al_version: Version):
al_version: Minimum imas_core version required for this command.
"""
used_version = ll_interface._al_version
if used_version >= al_version:
if used_version and used_version >= al_version:
return
click.echo(
f"This command requires at least version {al_version} of the Access Layer."
Expand Down
Loading