Skip to content

Conversation

mitchellkelly
Copy link

The package currently ignores embedded struct fields so patches wont get created for them. This PR adds the ability to walk embedded structs

@snorwin
Copy link
Owner

snorwin commented Nov 20, 2024

Hey @mitchellkelly, thanks for your contribution—it looks like an interesting use case! If you don’t mind, could you also extend the tests to cover it?

(In the meantime I will check the linter issues, I might have to upgrade the test framework)

@coveralls
Copy link

coveralls commented Nov 20, 2024

Pull Request Test Coverage Report for Build 16197735560

Details

  • 17 of 17 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.0%) to 92.184%

Totals Coverage Status
Change from base Build 14835279926: 1.0%
Covered Lines: 401
Relevant Lines: 435

💛 - Coveralls

@mitchellkelly
Copy link
Author

@snorwin I added some tests! Let me know what you think

@mitchellkelly
Copy link
Author

@snorwin could you review the tests I added? I'm happy to make any necessary changes!

@snorwin snorwin mentioned this pull request Apr 11, 2025
@mitchellkelly
Copy link
Author

@snorwin I noticed you mentioned this branch in your coverage PR! Is it possible to merge this ticket in now?

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