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

Shorten split helper function import path #271

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

folmos-at-orange
Copy link
Member

No description provided.

@folmos-at-orange folmos-at-orange self-assigned this Oct 30, 2024
@folmos-at-orange folmos-at-orange force-pushed the shorten-helpers-import-path branch 7 times, most recently from ccb70b0 to be132dc Compare October 31, 2024 16:56
@folmos-at-orange folmos-at-orange force-pushed the shorten-helpers-import-path branch 5 times, most recently from c553416 to 4b8f666 Compare November 25, 2024 10:21
@folmos-at-orange folmos-at-orange force-pushed the shorten-helpers-import-path branch 2 times, most recently from 050baa3 to db14860 Compare December 3, 2024 16:51
@@ -53,6 +53,7 @@ jobs:
with:
repository: khiopsml/khiops-python-tutorial
path: doc/khiops-python-tutorial
ref: shorten-helpers-path
Copy link
Collaborator

@popescu-v popescu-v Dec 3, 2024

Choose a reason for hiding this comment

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

After approval of this PR, should merge shorten-helpers-path into main in khiops-python-tutorial, then merge the current PR.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, this is in a TMP commit.

@folmos-at-orange folmos-at-orange force-pushed the shorten-helpers-import-path branch 3 times, most recently from 0993c25 to 88700df Compare December 11, 2024 07:53
@folmos-at-orange folmos-at-orange force-pushed the shorten-helpers-import-path branch from 88700df to aede15b Compare December 19, 2024 18:05
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