Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

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

Already on GitHub? # to your account

Feature/podaac 4328 #3

Merged
merged 9 commits into from
May 17, 2022
Merged

Feature/podaac 4328 #3

merged 9 commits into from
May 17, 2022

Conversation

yenes56
Copy link
Contributor

@yenes56 yenes56 commented May 13, 2022

Ticket: PODAAC-4328

Description

  • change this module to be compliant with cumulus 11 message. schema

Overview of work done

code change to make this module compliant with cumulus 11 schema
Merge from develop

Overview of verification done

Integration testing
MODIS_A
JASON_CS_S6A_L2_AMR_RAD_NTC
L1B_LR_INTF (this one failed with moveGranule step failed - as expected)

Tested in SIT:

Explain how you tested in SIT, if applicable

PR checklist:

  • Linted
  • [] Unit tests
  • Addressed Snyk vulnerabilities
  • Updated changelog
  • Tested in SIT
  • Documentation / User-Guide Updated

See Pull Request Review Checklist for pointers on reviewing this pull request

Yen, David (398B-Affiliate) added 4 commits May 5, 2022 19:59
# Conflicts:
#	.github/workflows/build-pipeline.yml
#	builder/builder.py
#	builder/requirements.txt
# Conflicts:
#	CHANGELOG.md
#	src/main/java/gov/nasa/cumulus/metadata/aggregator/CMRRestClientProvider.java
#	src/main/java/gov/nasa/cumulus/metadata/aggregator/MetadataAggregatorLambda.java
Copy link
Member

@ngachung ngachung left a comment

Choose a reason for hiding this comment

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

@yenes56 please also bump the version in pom.xml. Thank you!

@ngachung ngachung merged commit 072b1b7 into develop May 17, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants