Skip to content

Make build_multi_table_dictionary_domain private by convention#396

Merged
popescu-v merged 1 commit into
devfrom
395-backport-v11
Apr 15, 2025
Merged

Make build_multi_table_dictionary_domain private by convention#396
popescu-v merged 1 commit into
devfrom
395-backport-v11

Conversation

@popescu-v

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

Copy link
Copy Markdown
Collaborator

Eliminate the eponymous public (by convention) function, deprecated in v10.

Partial backport of the relevant v10 change c942f3f.


TODO Before Asking for a Review

  • Rebase your branch to the latest version of dev (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
    • If there are any problems it is faster to iterate by building locally the API Docs

@popescu-v popescu-v self-assigned this Apr 15, 2025

@folmos-at-orange folmos-at-orange left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You can keep the docstring though.

Eliminate the eponymous public (by convention) function, deprecated in
v10.

Partial backport of the relevant v10 change c942f3f.
@popescu-v

Copy link
Copy Markdown
Collaborator Author

You can keep the docstring though.

OK, done.

@popescu-v popescu-v merged commit 645223c into dev Apr 15, 2025
@popescu-v popescu-v deleted the 395-backport-v11 branch April 15, 2025 12:40
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.

3 participants