Skip to content

Add a function to clone a template type - #276

Merged
knoxsp merged 3 commits into
masterfrom
clone_template_type
Sep 5, 2025
Merged

Add a function to clone a template type#276
knoxsp merged 3 commits into
masterfrom
clone_template_type

Conversation

@knoxsp

@knoxsp knoxsp commented Jul 30, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@pmslavin pmslavin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One issue with the omission of cr_date from the cloned template that manifests in an assertion failure in the test, otherwise all fine and clear.

Comment thread tests/templates/test_templates.py Outdated
Comment thread hydra_base/lib/template/__init__.py
Comment thread hydra_base/lib/template/__init__.py
Comment thread tests/templates/test_templates.py
@knoxsp
knoxsp dismissed pmslavin’s stale review September 5, 2025 10:39

All comments addressed.

@knoxsp
knoxsp merged commit 1d774f5 into master Sep 5, 2025
6 checks passed
@knoxsp
knoxsp deleted the clone_template_type branch September 5, 2025 10:39
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