Skip to content

Customize khiops python tutorial ref#392

Merged
popescu-v merged 2 commits intodev-v10from
customize-khiops-python-tutorial-ref
Apr 14, 2025
Merged

Customize khiops python tutorial ref#392
popescu-v merged 2 commits intodev-v10from
customize-khiops-python-tutorial-ref

Conversation

@popescu-v
Copy link
Copy Markdown
Collaborator

@popescu-v popescu-v commented Apr 14, 2025

Customize the Khiops Python tutorial reference tag. Thusly, we can have the tutorials build for V11 beta (default, on main), as well as for V10 (via specific 10.3.1.0 tag).


TODO Before Asking for a Review

  • Rebase your branch to the latest version of dev-v10 (or main for release PRs)
  • Make sure all CI workflows are green
  • Self-Review: Review "Files Changed" tab and fix any problems you find
  • API Docs (only if there are changes in docstrings, rst files or samples):
    • Check the docs build without warning: see the log of the API Docs workflow
    • [ ] Check that your changes render well in HTML: download the API Docs artifact and open index.html There are no changes to the documentation itself; checking that it builds without warnings suffices, as this shows that the correct khiops-python-tutorial revision has been checked out.
    • If there are any problems it is faster to iterate by building locally the API Docs

@popescu-v popescu-v changed the base branch from dev to dev-v10 April 14, 2025 13:13
@popescu-v popescu-v requested review from folmos-at-orange, nairbenrekia and tramora and removed request for folmos-at-orange and nairbenrekia April 14, 2025 13:18
@popescu-v popescu-v self-assigned this Apr 14, 2025
Copy link
Copy Markdown
Collaborator

@tramora tramora left a comment

Choose a reason for hiding this comment

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

Looks good to me

@popescu-v popescu-v merged commit 5fb0e75 into dev-v10 Apr 14, 2025
3 checks passed
@popescu-v popescu-v deleted the customize-khiops-python-tutorial-ref branch April 14, 2025 14:07
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