Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fields. Fix None values representation in childs of ListField, DictField.#4118

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
tomchristie merged 1 commit intoencode:masterfromkhasanovbi:master
May 16, 2016
Merged

Conversation

khasanovbi
Copy link
Contributor

None values in childs of DictField, ListField not check in representation.

@codecov-io
Copy link

Current coverage is91.40%

Merging#4118 intomaster will not change coverage

@@             master      #4118   diff @@==========================================  Files            51         51            Lines          5476       5476            Methods           0          0            Messages          0          0            Branches          0          0          ==========================================  Hits           5005       5005            Misses          471        471            Partials          0          0

Powered byCodecov. Last updated by788603e...b73da53

@tomchristietomchristie added this to the3.3.4 Release milestoneMay 16, 2016
@tomchristietomchristie merged commit768ae26 intoencode:masterMay 16, 2016
@tomchristie
Copy link
Member

Good stuff, thanks! 😄 👍
Could you link to the original issue in this thread, with arefs: #xxx.

@khasanovbi
Copy link
ContributorAuthor

khasanovbi commentedMay 17, 2016
edited
Loading

@tomchristie Thanks. Sorry, I don't know, that first need to do issue and make pull request without issue. Should I do it now?

@tomchristie
Copy link
Member

Nope not to worry, thought there was an existing issue, but not a problem if there isn't.

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

Successfully merging this pull request may close these issues.

3 participants
@khasanovbi@codecov-io@tomchristie

[8]ページ先頭

©2009-2025 Movatter.jp