Skip to content

Refactored methods to don't override __init__ from RingGeneric#52

Merged
tchellomello merged 1 commit into
masterfrom
refactor_init
Sep 30, 2017
Merged

Refactored methods to don't override __init__ from RingGeneric#52
tchellomello merged 1 commit into
masterfrom
refactor_init

Conversation

@tchellomello

@tchellomello tchellomello commented Sep 30, 2017

Copy link
Copy Markdown
Member

Fixed init() methods on all Ring classes to handle it correctly.
There was no need to override the RingGeneric method since all the others one will inherit the same method signature.

Fixes issue: #49

@tchellomello tchellomello added bug Label for issues that report a bug enhancement labels Sep 30, 2017
@tchellomello tchellomello added this to the v.0.1.5 milestone Sep 30, 2017
@tchellomello tchellomello self-assigned this Sep 30, 2017
@coveralls

coveralls commented Sep 30, 2017

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling ce5e573 on refactor_init into 95775cc on master.

1 similar comment
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling ce5e573 on refactor_init into 95775cc on master.

@tchellomello tchellomello merged commit 0a2f8a9 into master Sep 30, 2017
@tchellomello tchellomello deleted the refactor_init branch September 30, 2017 04:31
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Label for issues that report a bug enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants