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

feat: add run link #278

Merged
merged 1 commit into from
Nov 8, 2022
Merged

feat: add run link #278

merged 1 commit into from
Nov 8, 2022

Conversation

bormando
Copy link
Contributor

@bormando bormando commented Sep 7, 2022

Added additional log at the run completion so it'd be possible to open results right from the console output (especially useful for CI/CD runs).

Feature request: https://qase.canny.io/feature-requests/p/log-a-link-to-qase-test-run-in-playwright-test-runs

image

@CLAassistant
Copy link

CLAassistant commented Sep 7, 2022

CLA assistant check
All committers have signed the CLA.

@sonarcloud
Copy link

sonarcloud bot commented Sep 7, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@bormando
Copy link
Contributor Author

@1ivliev any idea when you'd be able to review/add this?

@1ivliev
Copy link
Contributor

1ivliev commented Sep 28, 2022

@bormando This is expected shortly, after the merge:
#275

@stale
Copy link

stale bot commented Oct 19, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Oct 19, 2022
@stale stale bot closed this Oct 26, 2022
@bormando
Copy link
Contributor Author

bormando commented Nov 7, 2022

@1ivliev @nl-kaidzen @n3r can we add it or include into #275? I really need this (and I suppose I'm not the only one since this is one of the core features) and wait is lasting for months already for such a simple change.

@n3r n3r reopened this Nov 8, 2022
@n3r n3r self-requested a review November 8, 2022 09:43
@n3r n3r self-assigned this Nov 8, 2022
@stale stale bot removed the wontfix This will not be worked on label Nov 8, 2022
@n3r n3r merged commit 65a10ff into qase-tms:master Nov 8, 2022
@sonarcloud
Copy link

sonarcloud bot commented Nov 8, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@n3r
Copy link
Contributor

n3r commented Nov 8, 2022

Hello @bormando!

I apologize for the delay with review. I have merged this PR.

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.

4 participants