Skip to content

Add support for custom code generators - #66

Merged
LM-robynka merged 13 commits into
mainfrom
issue-65-add-support-for-custom-code-generators
Oct 28, 2025
Merged

LM-robynka merged 13 commits into
mainfrom
issue-65-add-support-for-custom-code-generators

Conversation

@LM-robynka

Copy link
Copy Markdown
Contributor

Fixing #65

@sveinse sveinse 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.

There are some comments to improvements. The gist of the implementation looks good, but there are some minor items that needs to be fixed. Unittest is missing.

Comment thread src/objdictgen/node.py Outdated
Comment thread src/objdictgen/node.py Outdated
Comment thread src/objdictgen/node.py Outdated
Comment thread src/objdictgen/node.py Outdated
@LM-robynka
LM-robynka requested a review from sveinse October 23, 2025 08:27
Comment thread src/objdictgen/node.py
Comment thread tests/test_node.py
@LM-robynka
LM-robynka requested a review from sveinse October 23, 2025 11:51
Comment thread src/objdictgen/node.py Outdated
Comment thread src/objdictgen/node.py Outdated
@LM-robynka
LM-robynka requested a review from sveinse October 24, 2025 14:12
Comment thread src/objdictgen/node.py Outdated
Comment thread tests/test_generators/bad_generator.py
Comment thread tests/test_generators/generator.py
@LM-robynka
LM-robynka requested a review from sveinse October 27, 2025 18:19

@sveinse sveinse 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.

LGTM

@LM-robynka
LM-robynka merged commit 70e7577 into main Oct 28, 2025
0 of 2 checks passed
@sveinse
sveinse deleted the issue-65-add-support-for-custom-code-generators branch November 4, 2025 09:41
@sveinse

sveinse commented Nov 4, 2025

Copy link
Copy Markdown
Member

@ronnie-kaczynski-laerdal I see now that the PR was merged with a standard merge and not squash, so all your trials, experiments, debugging commits have made it to its history. This is a public repo, so that was a bit unfortunate with the extra mess 😟

sveinse pushed a commit that referenced this pull request Nov 4, 2025
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