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

Allow the case of initializing attributes that are set to init=False #32

Merged
merged 3 commits into from Dec 20, 2015

Conversation

@hynek
Copy link
Member

@hynek hynek commented Dec 19, 2015

This allows for clean initializer parameter lists while being able to
initialize attributes to default values.

This allows for clean initializer parameter lists while being able to
initialize attributes to default values.
@codecov-io
Copy link

@codecov-io codecov-io commented Dec 19, 2015

Current coverage is 100.00%

Merging #32 into master will not affect coverage as of cabb2c1

@@            master     #32   diff @@
======================================
  Files            7       7       
  Stmts          339     343     +4
  Branches        78      80     +2
  Methods          0       0       
======================================
+ Hit            339     343     +4
  Partial          0       0       
  Missed           0       0       

Review entire Coverage Diff as of cabb2c1

Powered by Codecov. Updated on successful CI builds.

hynek added a commit that referenced this pull request Dec 20, 2015
Allow the case of initializing attributes that are set to `init=False`
@hynek hynek merged commit d590a0e into master Dec 20, 2015
3 checks passed
3 checks passed
codecov/project 100.00% (target 100.00%)
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@hynek hynek deleted the better-init-false branch Dec 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants