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: extract and set the LayerCI env variables #16233

Merged
merged 3 commits into from
Apr 27, 2021

Conversation

bahmutov
Copy link
Contributor

@bahmutov bahmutov commented Apr 27, 2021

User facing changelog

Cypress Test Runner automatically grabs the environment variables for LayerCI when recording to the dashboard

Additional details

These variables from https://layerci.com/docs/layerfile-reference/build-env are necessary to correctly show the commit / build information on Cypress Dashboard and to correctly tie multiple CI machines running in parallel to the same build

PR Tasks

@bahmutov bahmutov requested a review from a team as a code owner April 27, 2021 16:02
@bahmutov bahmutov requested review from chrisbreiding and kuceb and removed request for a team April 27, 2021 16:02
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Apr 27, 2021

Thanks for taking the time to open a PR!

@bahmutov bahmutov changed the title Add layerci env vars feat: extract and set the LayerCI env variables Apr 27, 2021
@jennifer-shehane jennifer-shehane self-assigned this Apr 27, 2021
@cypress
Copy link

cypress bot commented Apr 27, 2021



Test summary

4029 0 52 1Flakiness 0


Run details

Project cypress
Status Passed
Commit a644293
Started Apr 27, 2021 5:30 PM
Ended Apr 27, 2021 5:42 PM
Duration 11:47 💡
OS Linux Debian - 10.8
Browser Firefox 77

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

Copy link
Member

@jennifer-shehane jennifer-shehane left a comment

Choose a reason for hiding this comment

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

Looks all good! 👍

@jennifer-shehane jennifer-shehane merged commit 29b803f into develop Apr 27, 2021
@cypress-bot cypress-bot bot mentioned this pull request Apr 27, 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.

LayerCI Integration
2 participants