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.

Sign up for GitHub

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

Jump to bottom

ci: deploy a preview of the test report after every PR update #814

Open
NoamGaash opened this issue Jun 17, 2024 · 3 comments · May be fixed by #824
Open

ci: deploy a preview of the test report after every PR update #814

NoamGaash opened this issue Jun 17, 2024 · 3 comments · May be fixed by #824
Assignees
@arielrahamim
Labels
ci improvements to our CI pipeline good first issue Good for newcomers tests

Comments

@NoamGaash
Copy link
Member

on every pull request our bot adds a nice comment:
image

it could be great to add a preview of the test report as well
@NoamGaash NoamGaash added ci improvements to our CI pipeline tests good first issue Good for newcomers labels Jun 17, 2024
@arielrahamim
Copy link

hello, do you mean the test results url from the validilate.yaml workflow?
like this?
image

@NoamGaash
Copy link
Member Author

@arielrahamim exactly!

@arielrahamim arielrahamim linked a pull request Jun 19, 2024 that will close this issue
Open
@arielrahamim
Copy link

@NoamGaash made a PR for this, lmk any input u have.
i'm new to this project and tried digging around to understand your preview/ci logic and based my code around that.

@NoamGaash NoamGaash linked a pull request Jun 19, 2024 that will close this issue
ci: added test results to ci preview #824
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arielrahamim arielrahamim

Labels
ci improvements to our CI pipeline good first issue Good for newcomers tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ci: added test results to ci preview arielrahamim/open-bus-map-search
2 participants
@NoamGaash @arielrahamim