csvsoundsystem / federal-treasury-api

The scraper, parser, and database creation scripts for Financial Management Service daily U.S. Treasury statements.
www.treasury.io/
106 stars 27 forks source link

Screencast about the post-merge hook #177

Closed tlevine closed 7 years ago

tlevine commented 11 years ago

As CSV, I want a screencast about how the post-merge hook works and why we don't need to commit table_schema.json.

I think nobody understands this right now.

tlevine commented 11 years ago

I'm going to hold off on this until the new query builder is ready. (#182)

mhkeller commented 7 years ago

182 is closed and @bdewilde is curious about the post merge hook. Do you still remember this one @tlevine

tlevine commented 7 years ago

Oh, actually, I am free now. I was off by a week. Open this, or I can call you.

Michael Keller writes:

----==_mimepart_58026415d1f70_53a73f85462cf2bc460630 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit

182 is closed and @bdewilde is curious about the post merge hook. Do you st

ill remember this one @tlevine

You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/csvsoundsystem/federal-treasury-api/issues/177#issuecomme nt-253997507 ----==_mimepart_58026415d1f70_53a73f85462cf2bc460630 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 7bit

#182< /a> is closed and @bdewilde is curious about the post merge hook. Do you still remember t his one @tlevine


You are receiving this because you were mentioned.
Reply to this email directly,
view it on GitHub, or mute the thread.

----==_mimepart_58026415d1f70_53a73f85462cf2bc460630--

tlevine commented 7 years ago

I didn't remember, but I checked the documentation. According to the readme, we should set up the git post-merge hook in utils/post-merge. This hook generates the table_schema.json.

Michael Keller writes:

----==_mimepart_58026415d1f70_53a73f85462cf2bc460630 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit

182 is closed and @bdewilde is curious about the post merge hook. Do you st

ill remember this one @tlevine

You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/csvsoundsystem/federal-treasury-api/issues/177#issuecomme nt-253997507 ----==_mimepart_58026415d1f70_53a73f85462cf2bc460630 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 7bit

#182< /a> is closed and @bdewilde is curious about the post merge hook. Do you still remember t his one @tlevine


You are receiving this because you were mentioned.
Reply to this email directly,
view it on GitHub, or mute the thread.

----==_mimepart_58026415d1f70_53a73f85462cf2bc460630--

mhkeller commented 7 years ago

@tlevine we're in the chatb room

mhkeller commented 7 years ago

run.sh runs every day and that pulls info. the post merge hook is for local development. the idea was that when you merge, it rebuilds the table schema file but thomas isn't sure the table scheme file worked. but it's not used for anything currently.