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 version pull request job #1597

Merged
merged 2 commits into from
Mar 2, 2021
Merged

Conversation

roryabraham
Copy link
Contributor

Details

Fixing these failures.

Fixed Issues

https://github.com/Expensify/Expensify/issues/155197

Tests

  • Test by merging

Tested On

  • Github

@roryabraham roryabraham requested a review from AndrewGable March 2, 2021 17:52
@roryabraham roryabraham requested a review from a team as a code owner March 2, 2021 17:52
@roryabraham roryabraham self-assigned this Mar 2, 2021
@botify botify requested review from tylerkaraszewski and removed request for a team March 2, 2021 17:52
create_pull_request:
runs-on: ubuntu-latest
needs: version
steps:
- uses: actions/checkout@5a4ac9002d0be2fb38bd78e4b4dbde5606d7042f
with:
fetch-depth: 0
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need the entire repo or just the branch? https://github.com/actions/checkout#checkout-a-different-branch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure, we might only need the feature branch and master

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think since we probably need those two branches, the simplest for now is just to fetch everything. We can optimize for speed later by doing something more custom like one of the solutions or proposals from actions/checkout#214

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's fetch all for now 👍

@AndrewGable AndrewGable merged commit 4e5b6ed into master Mar 2, 2021
@AndrewGable AndrewGable deleted the Rory-FixVersionPullRequest branch March 2, 2021 18:02
@botify
Copy link

botify commented Mar 2, 2021

@AndrewGable looks like this was merged without passing tests. Please add a note explaining why this was done or remove the Emergency label if this is not an emergency.

@AndrewGable
Copy link
Contributor

Merged to fix deploys

@github-actions github-actions bot locked and limited conversation to collaborators Mar 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants