Skip to content

Conversation

@TG1999
Copy link
Contributor

@TG1999 TG1999 commented Apr 13, 2022

Remove null from string based fields

Signed-off-by: Tushar Goel tushar.goel.dav@gmail.com

@TG1999 TG1999 changed the title Reference: https://github.com/nexB/vulnerablecode/issues/692 Remove null from string based fields Apr 13, 2022
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.

LGTM, but are we missing a migration may be?

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.

LGTM: please just add the missing migration and merge.

@TG1999 TG1999 force-pushed the remove_null_from_string_fields branch 4 times, most recently from af46769 to 08c1e7e Compare April 15, 2022 19:07
Remove null from string based fields

Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
@TG1999 TG1999 force-pushed the remove_null_from_string_fields branch from 08c1e7e to c01b54b Compare April 15, 2022 19:11
@TG1999 TG1999 merged commit d1f2198 into aboutcode-org:main Apr 15, 2022
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.

2 participants