search

ceos-org / ceos-ard

Repository for CEOS Analysis Ready Data (CEOS-ARD), including Product Family Specifications (PFSs)
9 stars 1 forks source link

Add contributor information #36

Closed m-mohr closed 1 week ago

m-mohr commented 3 months ago

Fixes #28

gamedaygeorge commented 3 months ago

@m-mohr I ran through the steps in CONTRIBUTING.md, and they worked well, generating the attached. The only issue I had was a local configuration issue where I had to use a sudo to run the npm install command. I don't think this would be universal.

One question I had is on this:

You can now run the command to create the Word file in a Docker container

I was confused by the reference to a Docker container. Does pandoc run in Docker?

CEOS-ARD_PFS_Aquatic-Reflectance_latest.docx

m-mohr commented 3 months ago

The only issue I had was a local configuration issue where I had to use a sudo to run the npm install command. I don't think this would be universal.

I assume the same, but if anyone else on *nix machines runs into the issue, it could be worth mentioning.

I was confused by the reference to a Docker container. Does pandoc run in Docker?

Good catch, that's a left over from a first version where I used docker, but then I switched to just using Pandoc. I'll update the sentence accordingly. Thanks.