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

Add an additional column with the test name. #17

Conversation

erasmomontes
Copy link
Collaborator

🎉 New feature

Closes #10

Summary

  • Those changes add a new column in the summary with the name of the simulation command.

Test it

  • When CI passes it should insert a new summary in this PC. Check if it includes a new column called SimulationTest

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if it affects the public API)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@erasmomontes erasmomontes added the enhancement New feature or request label Aug 21, 2023
@github-actions
Copy link

Your image has been successfully uploaded to the remote repository. Please see the details below.

Tag: PR-17-9cee3ca-11
URL: 116476964151.dkr.ecr.us-east-1.amazonaws.com/cloud_robotic_devops_demo:PR-17-9cee3ca-11
AWS Respository URL: PR-17-9cee3ca-11

@github-actions
Copy link

Test results: 100.0 % | (4/4)

SimulationTest Sim-job Status Logs
✔️ test_set_initial_pose sim-gc0sk4hq191l SUCCESS Cloudwatch
✔️ test_set_move_to_pose sim-802hkgdqw4fy SUCCESS Cloudwatch
✔️ test_robot_moves sim-3h3l8417q1kf SUCCESS Cloudwatch
✔️ test_sensor_verification sim-rc8k58vb6gpz SUCCESS Cloudwatch

Copy link
Collaborator

@Voldivh Voldivh left a comment

Choose a reason for hiding this comment

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

LGTM

@erasmomontes erasmomontes merged commit c8944c4 into main Aug 21, 2023
5 checks passed
@erasmomontes erasmomontes deleted the 10-add-an-additional-column-with-the-test-name-to-the-github-action-report-on-the-pr branch August 21, 2023 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add an additional column with the test name to the github-action report on the PR
2 participants