Skip to content

[#698] move definition of IRODS_VERSION #784

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

Open
d-w-moore wants to merge 5 commits into
base: main
Choose a base branch
from
Open

[#698] move definition of IRODS_VERSION #784

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3963c67
[_698] move definition of IRODS_VERSION
d-w-moore Nov 25, 2025
b2c1e6a
Revert "[_698] move definition of IRODS_VERSION"
d-w-moore Jan 8, 2026
dbf5bd8
prefix underline to IRODS_VERSION symbol in irods.message namespace
d-w-moore Jan 8, 2026
1aed89d
comments to explain _IRODS_VERSION
d-w-moore Jan 8, 2026
bb05ba8
recreate _IRODS_VERSION as function returning server compatibility tuple
d-w-moore Jan 9, 2026
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: 1 addition & 0 deletions irods/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import sys

minimum_compatible_python = (3, 6)
maximum_compatible_server_api = (5, 0, 2, 'd')

if sys.version_info < minimum_compatible_python:
to_dotted_string = lambda version_tuple: ".".join(str(_) for _ in version_tuple)
Expand Down
2 changes: 1 addition & 1 deletion irods/message/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ def ET(xml_type=(), server_version=None):

logger = logging.getLogger(__name__)

IRODS_VERSION = (5, 0, 2, "d")
Copy link
Contributor

@korydraughn korydraughn Jan 8, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

Noting here that we cannot remove symbols from the public interface until a major version bump.

Steps:

  • Restore the old symbol for backward compatibility
  • Introduce _IRODS_VERSION as a duplicate of IRODS_VERSION
  • Dependent code must then be updated to use the new symbol
  • Add a comment stating the old symbol is deprecated

from .. import maximum_compatible_server_api as IRODS_VERSION

UNICODE = str

Expand Down