Skip to content

(VDB-1218) Add function for inserting headers#144

Merged
rmulhol merged 1 commit intostagingfrom
vdb-1218-no-duplicate-headers
Mar 17, 2020
Merged

(VDB-1218) Add function for inserting headers#144
rmulhol merged 1 commit intostagingfrom
vdb-1218-no-duplicate-headers

Conversation

@rmulhol
Copy link
Copy Markdown
Contributor

@rmulhol rmulhol commented Mar 11, 2020

  • Compatibility with upstream vdb changes

- Compatibility with upstream vdb changes

COMMENT ON TABLE maker.osm_change IS 'Note event emitted when change is called on OSM/PIP contract(s)';


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.

Idk why this is showing up as part of the PR but seems like maybe a migration was updated without the schema getting a corresponding update as we removed comments?

github.com/jmoiron/sqlx v0.0.0-20181024163419-82935fac6c1a
github.com/magiconair/properties v1.8.0
github.com/makerdao/vulcanizedb v0.0.13-rc.2
github.com/makerdao/vulcanizedb v0.0.13-rc.2.0.20200309155515-0dcc44e856be
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.

will switch back to staging and/or a release if we get it merged on the vdb side

Copy link
Copy Markdown
Contributor

@gslaughl gslaughl left a comment

Choose a reason for hiding this comment

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

:shipit:

@rmulhol rmulhol merged commit 11345f7 into staging Mar 17, 2020
@rmulhol rmulhol deleted the vdb-1218-no-duplicate-headers branch March 17, 2020 14:23
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