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

update(ci): always publish allure results and test results comment #869

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

luisecm
Copy link
Contributor

@luisecm luisecm commented Nov 20, 2024

What this PR does 📖

  • If tests were failing, allure report and automation results comment was not published. Fixing this by adding if always clause on the job

Which issue(s) this PR fixes 🔨

  • Resolve #

Special notes for reviewers 🗒️

Additional comments 🎤

@github-actions github-actions bot added the Missing dev review Two dev reviews are required on PR label Nov 20, 2024
@stavares843 stavares843 removed the Missing dev review Two dev reviews are required on PR label Nov 20, 2024
@luisecm luisecm merged commit 73c4409 into dev Nov 20, 2024
9 checks passed
@luisecm luisecm deleted the luis/fix-publish-report-nov-20 branch November 20, 2024 20:10
Copy link

github-actions bot commented Nov 20, 2024

Download the app installers for this pull request:

Copy link

Automated tests execution is complete! You can find the Playwright test report here and the Allure Test Report here

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