Skip to content

feat: replace dashboardUrl by resultsUrl#916

Merged
chrisbreiding merged 2 commits intocypress-io:masterfrom
MikeMcC399:add/cloudRunUrl
May 22, 2023
Merged

feat: replace dashboardUrl by resultsUrl#916
chrisbreiding merged 2 commits intocypress-io:masterfrom
MikeMcC399:add/cloudRunUrl

Conversation

@MikeMcC399
Copy link
Copy Markdown
Collaborator

@MikeMcC399 MikeMcC399 commented May 19, 2023

Closes #909.

This PR adds a GitHub step output cloudRunUrl resultsUrl for recording, providing a name which is not tied to the branding name of Cypress Dashboard or Cypress Cloud. This is a replacement for dashboardUrl.

The GitHub step output dashboardUrl, stemming from the previous Cypress Dashboard branding, continues to function and is documented as deprecated.

Documentation and examples are updated accordingly.

examples/v9 are left using dashboardUrl

@cypress-app-bot
Copy link
Copy Markdown

@chrisbreiding
Copy link
Copy Markdown

Let's go with resultsUrl for the name of the parameter.

@MikeMcC399 MikeMcC399 changed the title feat: replace dashboardUrl by cloudRunUrl feat: replace dashboardUrl by resultsUrl May 22, 2023
@MikeMcC399 MikeMcC399 marked this pull request as ready for review May 22, 2023 14:06
@MikeMcC399
Copy link
Copy Markdown
Collaborator Author

MikeMcC399 commented May 22, 2023

@chrisbreiding

Let's go with resultsUrl for the name of the parameter.

Thanks for the decision on the name. I've updated the PR contents accordingly. There are still references to the "Cypress Cloud" in text presented to users. This makes sense in terms of relating the parameter to other documentation.

(The branch name no longer reflects the name of the parameter. To change this would have required closing this PR and submitting a new one. Once the PR is merged I would delete the branch in this fork in any case.)

Comment thread index.js Outdated
@MikeMcC399

This comment was marked as outdated.

@chrisbreiding
Copy link
Copy Markdown

Thanks, @MikeMcC399!

@chrisbreiding chrisbreiding merged commit 1911878 into cypress-io:master May 22, 2023
@MikeMcC399 MikeMcC399 deleted the add/cloudRunUrl branch May 22, 2023 19:29
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 5.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace dashboardUrl using a Cypress Cloud oriented name

5 participants