Skip to content
This repository has been archived by the owner on Oct 6, 2022. It is now read-only.

Add "IPDS_id" field to the gran tsv? #11

Open
jordansread opened this issue Feb 28, 2015 · 4 comments
Open

Add "IPDS_id" field to the gran tsv? #11

jordansread opened this issue Feb 28, 2015 · 4 comments

Comments

@jordansread
Copy link
Member

Could be either a record ID (e.g., 061003 for laketemps) or "none" (or NA?)

Will help support future checks for .onAttach disclaimer. Also, will help address concerns from @rmhirsch49

@lawinslow
Copy link
Collaborator

We could also think about adding functionality to grantools. It wouldn't be hard to have it inject the disclaimer on any package not part of a "IPDS approved" list.

@lawinslow
Copy link
Collaborator

Wow, I had been watching your laketemps stuff come through, I didn't realize this was on grantools. So let me rephrase that, good idea. Could either check or just inject disclaimer as needed.

@jordansread
Copy link
Member Author

would be nice to inject if needed. that makes it so we wouldn't have to show everyone how to do it.

@lawinslow
Copy link
Collaborator

yeah, I know exactly where it would go in the workflow. The downloads from Github are uncompressed before being built. That's the moment to add the file. Would get us most of the way there.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants