Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Produce import_metadata_mcf in executor #1144

Merged
merged 4 commits into from
Dec 19, 2024
Merged

Produce import_metadata_mcf in executor #1144

merged 4 commits into from
Dec 19, 2024
+105 −20

Conversation

n-h-diaz
Copy link
Contributor

Tested manually: gs://datcom-prod-imports/scripts/us_fed/treasury_constant_maturity_rates/USFed_ConstantMaturityRates_Test/import_metadata_mcf.mcf

Output will be in "import_metadata_mcf.mcf"

@n-h-diaz n-h-diaz requested a review from ajaits December 16, 2024 20:24
ajaits
ajaits approved these changes Dec 17, 2024
View reviewed changes
Copy link
Contributor

@ajaits ajaits left a comment

Choose a reason for hiding this comment

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

Thanks Natalie!

We'll remove the lastDataRefreshDate and nextDataRefreshDate from autorefresh import provenance once this is integrated.

@n-h-diaz
Copy link
Contributor Author

Thanks Natalie!

We'll remove the lastDataRefreshDate and nextDataRefreshDate from autorefresh import provenance once this is integrated.

Thanks for the review! It looks like I don't have merge authorization for this repo, so please feel free to merge whenever

@ajaits ajaits merged commit f8c47fb into datacommonsorg:master Dec 19, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajaits ajaits ajaits approved these changes

Assignees

@ajaits ajaits

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@n-h-diaz @ajaits