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.

Sign up for GitHub

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

Jump to bottom

Wip add property accessor #16

Merged
merged 1 commit into from Sep 11, 2019
Merged

Wip add property accessor #16

merged 1 commit into from Sep 11, 2019

Conversation

eljam
Copy link
Owner

@eljam eljam commented Sep 10, 2019

No description provided.

@eljam eljam mentioned this pull request Sep 10, 2019
Closed
@coveralls
Copy link

coveralls commented Sep 10, 2019 •
edited

Coverage Status

Coverage remained the same at 100.0% when pulling 488f5da on add-accessor-property into c8ea177 on master.

@eljam eljam force-pushed the add-accessor-property branch 4 times, most recently from 40a500f to ea9eade Compare September 11, 2019 08:41
@eljam eljam merged commit 488f5da into master Sep 11, 2019
@eljam eljam deleted the add-accessor-property branch September 11, 2019 08:51
@nreynis
Copy link
Contributor

nreynis commented Sep 16, 2019

I'm sorry I won't be able to pull your code and test it. But it seems good to me. I guess you can close the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@eljam @coveralls @nreynis