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.

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

Feature/podaac-4659: Update failure and success association handling #43

Merged
merged 12 commits into from
Aug 17, 2022

Conversation

Phoeneix
Copy link
Contributor

Description

Update github concise repo on success and fail of automated testing. Similar to how l2ss-py automated testing is setup.

Overview of work done

  • removed branch restriction for update
  • updated launchpad token header for remove associations

Overview of verification done

Executed workflow
Failure check:
https://github.com/podaac/concise/runs/7868556797

Success check:
https://github.com/podaac/concise/runs/7867870614

Overview of integration done

Explain how this change was integration tested. Provide screenshots or logs if appropriate. An example of this would be a local Harmony deployment.

PR checklist:

  • Linted
  • Updated unit tests
  • Updated changelog
  • Integration testing

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

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

C2205121394-POCLOUD
Copy link
Contributor

Choose a reason for hiding this comment

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

So is this github action fully running now with updating the txt files on success or updating cmr on fails? if it is working lets not update these text files and let the github action workflow do it for us as a test.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This commit was created by the workflow automatically.

@Phoeneix Phoeneix merged commit ced685f into develop Aug 17, 2022
@Phoeneix Phoeneix deleted the feature/podaac-4659 branch August 17, 2022 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants