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

Show PR name on cypress dashboard #940

Closed
wants to merge 1 commit into from

Conversation

maiaht
Copy link
Contributor

@maiaht maiaht commented Dec 3, 2021

We did not manage to figure out how to do this, so this PR will be closed. For more details see the comment below.

@maiaht maiaht self-assigned this Dec 3, 2021
@maiaht maiaht marked this pull request as ready for review December 3, 2021 09:03
@maiaht maiaht linked an issue Dec 6, 2021 that may be closed by this pull request
@maiaht maiaht force-pushed the format-dashboard branch 6 times, most recently from dc9c932 to d44ff53 Compare December 7, 2021 09:39
@maiaht maiaht marked this pull request as draft December 8, 2021 09:08
@maiaht maiaht force-pushed the format-dashboard branch 10 times, most recently from bfc9219 to 3732635 Compare December 15, 2021 09:49
@@ -59,6 +59,8 @@ jobs:
displayName: "Start frontend"

- script: |
COMMIT_INFO_MESSAGE=$(git log --oneline -2 --pretty=format:%s | tail -1)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The content of line 62 has to be placed under script. The problem is that setting the COMMIT_INFO_MESSAGE in the script section does not work. The COMMIT_INFO_MESSAGE has to be an environment variable placed under the env section, however, in env we have not managed to set an environment variable to a value that has been set under the script section.

@maiaht maiaht closed this Dec 16, 2021
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.

Display formatted title in Cypress for each pr instead of hash
2 participants