-
Notifications
You must be signed in to change notification settings - Fork 76
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
@W-14913383 | Fix e2e pipeline #302
Conversation
.github/workflows/e2etest.yml
Outdated
@@ -1,61 +1,61 @@ | |||
name: e2e-tests | |||
|
|||
on: | |||
pull_request: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there are so many changes due to formatting, can u describe the actual change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kalagarraj Sure, thank you for asking. In the step "Auth Devhub" I have just changed
echo ${{ env.DEVHUB_AUTH }} > authFile
sf force auth sfdxurl store -f 'authFile'
to
echo '${{ env.DEVHUB_AUTH }}' > authFile
sf force auth sfdxurl store -f authFile
this fixed the inconsistency happened in some PR's where the authFile was not read correctly.
.github/workflows/e2etest.yml
Outdated
PR_NUMBER: ${{ github.event.pull_request.number }} | ||
steps: | ||
- name: Checkout |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
something is not correct in new formatting... did u use any linter or IDE to reformat?
this new one has 8 spaces where as the old one had 4 spaces for indentation. I think u might consider moving back to 4 spaces for indentation
What does this PR do?
Trying to fix the e2e test
@W-14913383@