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

test: run component test example from current branch #1013

Merged
merged 1 commit into from Aug 30, 2023

Conversation

MikeMcC399
Copy link
Collaborator

@MikeMcC399 MikeMcC399 commented Aug 30, 2023

This PR changes the workflow .github/workflows/example-component-test.yml to run the action from the HEAD of the branch it is started in instead of explicitly from the v5 branch as is currently the case:

- name: Cypress run
uses: cypress-io/github-action@v5

This aligns the workflow with all the other example workflows in .github/workflows which use the construct:

      - name: Cypress tests
        # normally you would write
        # uses: cypress-io/github-action@v5
        uses: ./

@cypress-app-bot
Copy link

@MikeMcC399 MikeMcC399 marked this pull request as ready for review August 30, 2023 10:34
@nagash77 nagash77 merged commit 02c1997 into cypress-io:master Aug 30, 2023
68 checks passed
@MikeMcC399 MikeMcC399 deleted the component-test-from-current branch August 30, 2023 17:17
@github-actions
Copy link

github-actions bot commented Sep 1, 2023

🎉 This PR is included in version 6.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

ax-vasquez added a commit that referenced this pull request Sep 1, 2023
* WIP - need to ask for ideal way to check existing variables

* check for existing variables

* consolidate comments

* remove yarn lock file

* run commands from PR feedback

* logging for testing purposes only

* more logging

* more logging

* simplify URL logic

* more logging

* remove logging - fix url string

* log response for fallback logic

* docs: rewrite readme working directory description (#1005)

* chore(deps): update dependency @types/node to v20.5.7 (#1008)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* docs: rework requirements for contributors (#1010)

* feat(deps): update cypress to 13.0.0 (#1012)

* logging for testing purposes only

* test: run component test example from current branch (#1013)

* more logging

* more logging

* simplify URL logic

* more logging

* remove logging - fix url string

* log response for fallback logic

* cleanup

* doc update

* Update README.md

Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>

* Update README.md

Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>

* exit detectPrNumber if variables already defined

* Update README.md

Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>

* docs update - formatting fix for readability

* Update README.md

Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>

* Update README.md

Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>

* try catch pr data request

* Readme update

* Update README.md

Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>

---------

Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

None yet

3 participants