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

fields_count was remake by value based approach #101

Closed
wants to merge 2 commits into from

Conversation

denzor200
Copy link
Contributor

This approach gives us ability for make overloads of fields_count function.
And this ability will help me with #100

@coveralls
Copy link

Pull Request Test Coverage Report for Build 2832143145

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 1768569863: 0.0%
Covered Lines: 332
Relevant Lines: 332

💛 - Coveralls

@denzor200
Copy link
Contributor Author

Im not sure. It will be reopened if we really need it

@denzor200 denzor200 closed this Aug 12, 2022
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