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

fix(ci): setup global git user #2157

Merged
merged 3 commits into from
Nov 7, 2023
Merged

Conversation

szymonrybczak
Copy link
Collaborator

Summary:

Sometimes initialising git repository fails, because machine doesn't have git installed or configured correctly so when running E2E tests we need to take this into account and dynamically assign files that we're expecting.

Test Plan:

CI

Checklist

  • Documentation is up to date to reflect these changes.
  • Follows commit message convention described in CONTRIBUTING.md

@thymikee
Copy link
Member

thymikee commented Nov 6, 2023

Let's set the user in CI config, should allow us to not modify the test code

@szymonrybczak szymonrybczak force-pushed the fix/dynamically-assign-expected-files branch from ac4fac1 to 8dbef75 Compare November 6, 2023 09:56
@szymonrybczak szymonrybczak changed the title fix(e2e): dynamically assign expected files if initialising git repo failed fix(ci): setup git user Nov 6, 2023
@szymonrybczak szymonrybczak changed the title fix(ci): setup git user fix(ci): setup global git user Nov 6, 2023
@szymonrybczak szymonrybczak force-pushed the fix/dynamically-assign-expected-files branch from af246d8 to 80870fd Compare November 6, 2023 16:34
@szymonrybczak szymonrybczak force-pushed the fix/dynamically-assign-expected-files branch from 80870fd to 8590b80 Compare November 6, 2023 16:45
@szymonrybczak szymonrybczak force-pushed the fix/dynamically-assign-expected-files branch from 4be15b0 to 668b13f Compare November 6, 2023 17:15
@szymonrybczak
Copy link
Collaborator Author

Let's set the user in CI config, should allow us to not modify the test code

Done, CI Green ✅

@@ -47,6 +47,9 @@ jobs:
node-version: [18, 20]
os: [ubuntu-latest, macos-latest, windows-2019]
runs-on: ${{ matrix.os }}
defaults:
run:
shell: bash
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It needs to be specified, since by default it launches Powershell on Windows.

@thymikee thymikee merged commit 918e3e0 into main Nov 7, 2023
@thymikee thymikee deleted the fix/dynamically-assign-expected-files branch November 7, 2023 09:37
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.

2 participants