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

Add more test cases for subclasses using @field #83

Merged
merged 2 commits into from
Jan 25, 2023
Merged

Conversation

BurnzZ
Copy link
Contributor

@BurnzZ BurnzZ commented Sep 28, 2022

Adding and fixing some use cases before Approach 1 from #77 begins development.

@BurnzZ BurnzZ changed the title WIP: Fix subclass overriding field from parent class using to_item() WIP: Add more test cases for subclasses using @field Sep 28, 2022
@codecov
Copy link

codecov bot commented Sep 28, 2022

Codecov Report

Merging #83 (6736287) into master (34eb0ec) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #83   +/-   ##
=======================================
  Coverage   99.81%   99.81%           
=======================================
  Files          17       17           
  Lines         536      536           
=======================================
  Hits          535      535           
  Misses          1        1           

@kmike
Copy link
Member

kmike commented Jan 23, 2023

@BurnzZ should we merge or close the PR?

@BurnzZ BurnzZ marked this pull request as ready for review January 24, 2023 02:01
@BurnzZ BurnzZ changed the title WIP: Add more test cases for subclasses using @field Add more test cases for subclasses using @field Jan 24, 2023
@BurnzZ
Copy link
Contributor Author

BurnzZ commented Jan 24, 2023

@kmike , I forgot about this, thanks for reminding. I think we could still merge it after reviewing.

@BurnzZ BurnzZ requested review from Gallaecio and kmike January 24, 2023 02:02
@kmike kmike merged commit f7e5be2 into master Jan 25, 2023
@kmike kmike deleted the fix-some-use-cases branch January 25, 2023 14:27
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