nasa / cmr-stac

Other
55 stars 21 forks source link

CMR-10184: Add link to collection if a STAC API is present in the related urls of collection metadata. #356

Closed doug-newman-nasa closed 1 month ago

doug-newman-nasa commented 1 month ago

https://bugs.earthdata.nasa.gov/browse/CMR-10184

A CMR collection can now indicate to consumers that it has a STAC API. If that is the case then we should use that link instead of a generic one inserted at runtime. This is useful of collections that do not index their granule metadata in CMR, like CWIC collections.

codecov-commenter commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.34%. Comparing base (0590a70) to head (78eb551).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #356 +/- ## ========================================== + Coverage 88.19% 88.34% +0.15% ========================================== Files 24 24 Lines 1152 1167 +15 Branches 250 256 +6 ========================================== + Hits 1016 1031 +15 Misses 136 136 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

doug-newman-nasa commented 1 month ago

Question: how do you run the prettier on the command line? I tried the ' --write' option but it only told me which files need to be changed. I resorted to installing prettier on VS code and formatting each file to get past the CI check. I'd rather do it at the command line and add documentation for others.

dmistry1 commented 1 month ago

Question: how do you run the prettier on the command line? I tried the ' --write' option but it only told me which files need to be changed. I resorted to installing prettier on VS code and formatting each file to get past the CI check. I'd rather do it at the command line and add documentation for others.

I typically I run the command npm run prettier:fix, which essentially runs prettier src --write to resolve formatting issues.