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

adding initial support for constructors #23

Merged
merged 5 commits into from
Jun 16, 2016
Merged

adding initial support for constructors #23

merged 5 commits into from
Jun 16, 2016

Conversation

zabawaba99
Copy link
Owner

No description provided.


private Object obj;

public Constructors(Object obj) {
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make this private

SampleOne class does not have a constructor with parameters
@coveralls
Copy link

coveralls commented Jun 16, 2016

Coverage Status

Coverage decreased (-3.03%) to 96.97% when pulling 9ff8530 on constructors into 7c12802 on master.

Fetching a constructor by name doesn't really make too much sense. Usually, you would be getting declared constructors so the name would always be the same
@coveralls
Copy link

coveralls commented Jun 16, 2016

Coverage Status

Coverage decreased (-3.1%) to 96.899% when pulling aad6e17 on constructors into 7c12802 on master.

@zabawaba99 zabawaba99 merged commit c6d4da7 into master Jun 16, 2016
@zabawaba99 zabawaba99 deleted the constructors branch June 16, 2016 02:05
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.

None yet

2 participants