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

Reverted Changes to Git Config and Authentication #64

Merged
merged 13 commits into from
Nov 21, 2019

Conversation

thboop
Copy link
Contributor

@thboop thboop commented Nov 4, 2019

The updates were breaking user's builds #60, #62.

@thboop thboop force-pushed the users/thboop/revertCheckoutUpdates branch from 4e62d6b to 46c560d Compare November 4, 2019 21:57
@thboop
Copy link
Contributor Author

thboop commented Nov 5, 2019

Resolves #62

@@ -15,4 +15,5 @@ jobs:
steps:
- uses: actions/checkout@master
- uses: ./
- run: git ls-remote --tags origin
with:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We had a flow to test the v1.2 feature, so we need to update our tests on the revert

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@thboop thboop force-pushed the users/thboop/revertCheckoutUpdates branch from 61aa855 to eb30a95 Compare November 20, 2019 18:53
README.md Outdated
```yaml
- uses: actions/checkout@v1
with:
submodules: recursive
submodules: true // 'recursive' 'true' or 'false'
Copy link
Contributor

Choose a reason for hiding this comment

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

#

@thboop thboop merged commit 50fbc62 into master Nov 21, 2019
@thboop thboop deleted the users/thboop/revertCheckoutUpdates branch November 21, 2019 16:04
@Imebeez
Copy link

Imebeez commented Aug 20, 2024

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.

4 participants