-
-
Notifications
You must be signed in to change notification settings - Fork 176
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
Add support for updating the current HEAD #204
Conversation
update.sh: fatal: Refusing to fetch into current branch refs/heads/master of non-bare repository error fix
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.
Thanks for implementing a fix. Could you include a test that covers your changes.
@talva-tr Thanks for making this fix have you tried using push event or switching back to the current PR branch before running this action ?? |
Hey, I tried switching back to the current PR branch and it didn't work, so
I forked the repo and fixed it
…On Mon, Oct 11, 2021 at 5:42 PM Tonye Jack ***@***.***> wrote:
@talva-tr <https://github.com/talva-tr> Thanks for making this fix have
your tried using push event or switching back to the current PR branch
before running this action ??
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#204 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ATR7ABJFXHFFHHZKASOVCT3UGLZUHANCNFSM5FYGDXKA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
@talva-tr Can you also post a sample of your workflow file |
on:
push:
branches:
- main
paths:
- '!.github/**'
jobs:
build:
runs-on: self-hosted
steps:
- name: checkout the python repo
uses: ***@***.***
with:
fetch-depth: 2
- name: Get changed files
id: changed-files
uses: ***@***.***
- name: Set up Python 3.9
uses: ***@***.***
with:
python-version: 3.9
- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install elasticsearch requests
- name: run validation script
run: |
python3 e.py "${{ steps.changed-files.outputs.all_modified_files }}"
…On Tue, Oct 12, 2021 at 2:05 PM Tonye Jack ***@***.***> wrote:
Hey, I tried switching back to the current PR branch and it didn't work, so
I forked the repo and fixed it
On Mon, Oct 11, 2021 at 5:42 PM Tonye Jack *@*.***> wrote:
@talva-tr <https://github.com/talva-tr> https://github.com/talva-tr
Thanks for making this fix have
your tried using push event or switching back to the current PR branch
before running this action ??
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#204 (comment)
<#204 (comment)>
,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATR7ABJFXHFFHHZKASOVCT3UGLZUHANCNFSM5FYGDXKA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.
@talva-tr <https://github.com/talva-tr> Can you also post a sample of
your workflow file
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#204 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ATR7ABK6Z4ZTGZ3W3ML6U5TUGQI7BANCNFSM5FYGDXKA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
@all-contributors add @talva-tr for code |
I've put up a pull request to add @talva-tr! 🎉 |
@talva-tr Thanks for the fix this should be available in the latest release. |
Thank you.
…On Wed, Oct 13, 2021 at 2:35 PM Tonye Jack ***@***.***> wrote:
@talva-tr <https://github.com/talva-tr> Thanks for the fix this should be
available in the latest release.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#204 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ATR7ABJPCROJQ4KFK3L3W63UGVVHVANCNFSM5FYGDXKA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
update.sh: fatal: Refusing to fetch into current branch refs/heads/master of non-bare repository error fix