Skip to content

(VDB-1051) Create flop vow storage transformer#65

Merged
gslaughl merged 1 commit intostagingfrom
vdb-1051-flop-vow-transformer
Dec 30, 2019
Merged

(VDB-1051) Create flop vow storage transformer#65
gslaughl merged 1 commit intostagingfrom
vdb-1051-flop-vow-transformer

Conversation

@gslaughl
Copy link
Copy Markdown
Contributor

No description provided.

@rmulhol
Copy link
Copy Markdown
Contributor

rmulhol commented Dec 16, 2019

LGTM but withholding approval since this shouldn't be merged until 0.2.17 is cut 👍

Copy link
Copy Markdown
Contributor

@elizabethengelman elizabethengelman left a comment

Choose a reason for hiding this comment

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

🖖looks good!

I think this could probably get merged in whenever now that we've cut v0.2.17 🎉

Tau = "tau"
Kicks = "kicks"
Live = "live"
Vow = "vow"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

totally doesn't have to be part of this PR, but I wonder if we should rename this file? 🤔

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah good point. Seems like these vars might be more at home in constants/keys.go, with the rest of our key/column strings

@gslaughl gslaughl force-pushed the vdb-1051-flop-vow-transformer branch from 9e62ddc to 9d1c316 Compare December 30, 2019 21:01
@gslaughl gslaughl merged commit 8cb3464 into staging Dec 30, 2019
@gslaughl gslaughl deleted the vdb-1051-flop-vow-transformer branch December 30, 2019 21:13
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.

3 participants