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

DS-2940 setup unit tests with output processing, pr decoration #1436

Merged
merged 3 commits into from
Sep 24, 2024

Conversation

peterswirl
Copy link
Collaborator

@peterswirl peterswirl commented Sep 24, 2024

Automatically run the unit test suite on PR to develop branch. This will output a junit.xml and then decorate job run/related PRs with the report.

Branching Reminders

  • For core code changes, create a branch off develop
  • For product documentation changes, create a branch off main
  • Always name your branch in a way that clearly describes your proposed changes

Description

  • update perms for run so it can modify PR status
  • add trigger for pull requests to develop
  • output as junit xml and publish report

Related Issue(s)

N/A

Testing and Validation

Type of Change

  • Bug fix or other non-breaking change that addresses an issue
  • New Feature / Enhancement (non-breaking change that add or improves functionality)
  • New Feature (breaking change that is not backwards compatible and/or alters current functionality)
  • Documentation (change to product documentation or README.md only)

@peterswirl peterswirl self-assigned this Sep 24, 2024
Peter Kahn added 2 commits September 24, 2024 10:15
Revert "add newline to swirl/performance_logger.py to trigger test execution"

This reverts commit 4760be0.
@peterswirl peterswirl merged commit 7a9a9f2 into develop Sep 24, 2024
@peterswirl peterswirl deleted the feature/DS-2940-ut-on-merge branch September 24, 2024 14:39
peterswirl added a commit that referenced this pull request Sep 24, 2024
* setup unit tests with output processing, pr decoration

---------

Co-authored-by: Peter Kahn <[email protected]>
(cherry picked from commit 7a9a9f2)
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.

2 participants