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

Fix code generation for interfaces inheriting from other interfaces #36

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

HugoX104447
Copy link
Contributor

Context

This pull request addresses an issue in the code generation process for interfaces that inherit from other interfaces. The current implementation does not generate the code for the inherited interfaces.

Changes Made

  • Update the code generator to properly recognize and process inherited interfaces.
  • Add unit test to validate the changes.

@connorivy
Copy link
Owner

hey @HugoX104447,

thanks so much for the PR! I should have some time to take a look at it tomorrow. I can see that you added some tests so that should help me out a lot.

- Update the code generator to properly recognize and process inherited
  interfaces.
- Add unit test to validate the changes.
@HugoX104447 HugoX104447 force-pushed the fix-interface-inheritance branch from 5d682b6 to 637dc32 Compare January 23, 2025 06:43
@connorivy connorivy changed the base branch from main to dev January 23, 2025 21:12
@connorivy connorivy merged commit ebe61f5 into connorivy:dev Jan 23, 2025
2 of 3 checks passed
connorivy added a commit that referenced this pull request Jan 23, 2025
* Fix code generation for methods with default arguments

* Fix code generation for interfaces inheriting from other interfaces (#36)

- Update the code generator to properly recognize and process inherited
  interfaces.
- Add unit test to validate the changes.

* use unified interface and run formatter

* Fix code generation for methods with default arguments (#35)

* move default argument tests to shared project

* Fix code generation for interfaces with overloaded methods (#38)

* Fix code generation for interfaces inheriting from other interfaces

- Update the code generator to properly recognize and process inherited
  interfaces.
- Add unit test to validate the changes.

* Fix code generation for interfaces with overloaded methods

---------

Co-authored-by: Karsten Heimrich <[email protected]>
Co-authored-by: Karsten Heimrich <[email protected]>
@connorivy
Copy link
Owner

Hey @HugoX104447,

I merged this in and it's part of 1.1.0. Let me know if you have any issues with it.

@HugoX104447 HugoX104447 deleted the fix-interface-inheritance branch January 24, 2025 16:23
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