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 nested enums in meta #49

Merged
merged 2 commits into from
Jul 7, 2016
Merged

Conversation

sviperll
Copy link
Collaborator

@sviperll sviperll commented Jul 7, 2016

No description provided.

Nested enums caused an exception when processed through meta-helpers.
Fixed it for now.
@codecov-io
Copy link

codecov-io commented Jul 7, 2016

Current coverage is 36.22%

Merging #49 into master will decrease coverage by 0.05%

@@             master        #49   diff @@
==========================================
  Files           117        117          
  Lines          4874       4881     +7   
  Methods           0          0          
  Messages          0          0          
  Branches        973        979     +6   
==========================================
  Hits           1768       1768          
- Misses         2872       2879     +7   
  Partials        234        234          

Powered by Codecov. Last updated by faae724...037fbec

@phax phax merged commit 9e13eff into phax:master Jul 7, 2016
@phax
Copy link
Owner

phax commented Jul 7, 2016

Is this enough for a new release?

@sviperll
Copy link
Collaborator Author

sviperll commented Jul 7, 2016

I have no rush:) Personally I have no agenda for jcodemodel and plan no further changes in a mean time, so you can freely proceed with the release as you feel right.

@phax
Copy link
Owner

phax commented Jul 7, 2016

I got quite quiet in the last time so I think people are happy with the "as is" situation (or they changed to some library???). Anyway I will release it to indicate progress :)

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