Skip to content
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.

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

Keep track of version in library #98

Merged
merged 1 commit into from
Nov 10, 2021
Merged

Keep track of version in library #98

merged 1 commit into from
Nov 10, 2021

Conversation

galleon
Copy link
Contributor

@galleon galleon commented Nov 8, 2021

This PR does the following:

  • request poetry to change version in skdecide/__iniy__.py
  • print the information when utils/collect_env.pyis called

@dbarbier
Copy link
Contributor

dbarbier commented Nov 8, 2021

Please write in commit message the new version number, it is hard to guess.

@@ -18,6 +18,12 @@
except (ImportError, NameError, AttributeError, OSError):
TORCH_AVAILABLE = False

try:
from skdecide import __version__
Copy link
Contributor

Choose a reason for hiding this comment

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

You could write

try:
    from skdecide import __version__ as skdecide_version
 except (ImportError):
    skdecide_version = "not available"

and use skdecide_version directly

  - vx.y.z.dev{{distance}}+{{sha}} for dev release
dbarbier added a commit to dbarbier/scikit-decide that referenced this pull request Nov 9, 2021
The only open pull request which modifies files under skdecide/
directory is airbus#98, conflict resolution will be trivial, so we can apply
pre-commit linters.

There is no code change, this commit only contains changes performed
by running

    pre-commit run -a
@galleon galleon merged commit 5e84486 into airbus:master Nov 10, 2021
@galleon galleon deleted the gal/versioning branch November 20, 2021 11:44
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.

2 participants