Skip to content

Conversation

xavierfigueroav
Copy link
Collaborator

@xavierfigueroav xavierfigueroav commented Apr 17, 2019

Fix #102
This PR modifies config.py file to use the method from_str in Version instead of creating the object directly with string values.

Signed-off-by: Xavier Figueroa xavierfigueroav@gmail.com

@pombredanne
Copy link
Member

We would need some tests for this!

@xavierfigueroav
Copy link
Collaborator Author

We would need some tests for this!
@pombredanne There are some tests now!

Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

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

All good. Thank you!

@pombredanne
Copy link
Member

But this needs to be rebased too

Signed-off-by: Xavier Figueroa <xavierfigueroav@gmail.com>
Signed-off-by: Xavier Figueroa <xavierfigueroav@gmail.com>
…ion.json.

Add tests for exceptions list in config.py.

Signed-off-by: Xavier Figueroa <xavierfigueroav@gmail.com>
@xavierfigueroav
Copy link
Collaborator Author

All good. Thank you!

Great!

But this needs to be rebased too

@pombredanne It is now rebased!

@pombredanne
Copy link
Member

Thanks!

@xavierfigueroav xavierfigueroav deleted the version-fields-issue branch August 20, 2019 22:34
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.

Version model fields must be integers but sometimes they aren't
2 participants