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

Use for test job a macOS machine to be able to run golden tests #220

Merged
merged 4 commits into from
May 12, 2022

Conversation

nilsreichardt
Copy link
Member

#138 & #216 are adding golden tests. Golden tests can only run on one platform. We decided to pick macOS as the platform because primary using macOS. Therefore, should our CI use macOS machines. Otherwise, the Golden Tests are skipped.

@github-actions
Copy link

github-actions bot commented May 11, 2022

Visit the preview URL for this PR (updated for commit 2ad453a):

https://sharezone-test--pr220-use-macos-machine-fo-pjx6pjz7.web.app

(expires Wed, 18 May 2022 22:59:59 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

- run: echo $(realpath ./bin) >> $GITHUB_PATH
- run: echo $(pwd)/bin >> $GITHUB_PATH
Copy link
Collaborator

Choose a reason for hiding this comment

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

Out of curiosity: Why did you change this? Did realpath lead to problems?

@Jonas-Sander Jonas-Sander merged commit 0fba041 into main May 12, 2022
@Jonas-Sander Jonas-Sander deleted the use-macos-machine-for-test-job branch May 12, 2022 15:06
nilsreichardt added a commit that referenced this pull request May 17, 2022
## Description
This PR contains a proposal to display an alpha version banner.

Users which are using an alpha version should be aware that is an alpha version. Therefore, a small banner at the top right-hand corner indicates this.

This banner also helps us when somebody sends a screenshot of an alpha version. We can instantly see that is an alpha version.

When this proposal got accepted, we can add the environment variable to the `deploy_web_app` workflow.

## Demo
<img width="1135" alt="image" src="https://user-images.githubusercontent.com/24459435/167873877-2a0f512b-14c9-4ee9-a0f8-5662c5c26b52.png">

## Related Tickets
Related to #27

The PR #220 should be merged first.
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.

2 participants