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

Updating markdown tables to allow for missing datapoints #17

Merged
merged 5 commits into from
Aug 17, 2023

Conversation

stevematney
Copy link
Contributor

@stevematney stevematney commented Aug 17, 2023

Summary of PR changes

Allow for possible missing data on a Postman report in order to use the json-summary reporter rather than the json reporter which creates absurdly large artifacts

PR Requirements

  • For major, minor, or breaking changes, at least one of the commit messages contains the appropriate +semver: keywords.
    • See the Incrementing the Version section of the repository's README.md for more details.
  • The action code does not contain sensitive information.

NOTE: If the repo's workflow could not automatically update the README.md, it should be updated manually with the next version. For javascript actions, if the repo's workflow could not automatically recompile the action it should also be updated manually as part of the PR.

@stevematney stevematney requested a review from a team as a code owner August 17, 2023 20:27
@stevematney stevematney force-pushed the support-json-summary-report branch 2 times, most recently from 660e0b8 to 9a63a42 Compare August 17, 2023 21:15
@stevematney stevematney force-pushed the support-json-summary-report branch from 0cecda9 to 47d8c32 Compare August 17, 2023 21:18
@stevematney
Copy link
Contributor Author

Here's an example of the status check that was created. Looks correct!
https://github.com/im-enrollment/frontend-tooling-bff/pull/757/checks?check_run_id=15993149777

@mike-schenk mike-schenk merged commit 4cc90df into main Aug 17, 2023
@mike-schenk mike-schenk deleted the support-json-summary-report branch August 17, 2023 23:44
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