Skip to content

Disable "unable to get git repo" when none exists.#2495

Merged
mergify[bot] merged 3 commits intonektos:masterfrom
stephenwithav:set-nektos/act-as-repo-when-no-remote-exists-to-suppress-errors
Feb 25, 2025
Merged

Disable "unable to get git repo" when none exists.#2495
mergify[bot] merged 3 commits intonektos:masterfrom
stephenwithav:set-nektos/act-as-repo-when-no-remote-exists-to-suppress-errors

Conversation

@stephenwithav
Copy link
Contributor

Closes #2493.

@stephenwithav stephenwithav requested a review from a team as a code owner October 19, 2024 23:23
Copy link
Contributor

@ChristopherHX ChristopherHX left a comment

Choose a reason for hiding this comment

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

I overal agree about silencing this warning, in my use of act this code is effectively disabled.

Neutral, I approve if another maintainer approves

@codecov
Copy link

codecov bot commented Oct 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.43%. Comparing base (5a80a04) to head (77cd756).
Report is 182 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #2495       +/-   ##
===========================================
+ Coverage   61.56%   74.43%   +12.86%     
===========================================
  Files          53       72       +19     
  Lines        9002    11056     +2054     
===========================================
+ Hits         5542     8229     +2687     
+ Misses       3020     2191      -829     
- Partials      440      636      +196     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mergify mergify bot requested a review from a team October 20, 2024 09:50
Copy link
Contributor

@ChristopherHX ChristopherHX left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify
Copy link
Contributor

mergify bot commented Feb 25, 2025

This pull request has been removed from the queue for the following reason: pull request branch update failed.

The pull request can't be updated

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.

If you want to requeue this pull request, you need to post a comment with the text: @mergifyio requeue

@mergify mergify bot merged commit c6a7754 into nektos:master Feb 25, 2025
12 checks passed
project-mirrors-bot-tu bot pushed a commit to project-mirrors/forgejo-runner-as-gitea-act-runner-fork that referenced this pull request Aug 10, 2025
Refs nektos/act#2495

---

* Disable "unable to get git repo" when none exists.

* Restore error back so it's included in bug reports

Change from Warningf to Debugf.

---------

Co-authored-by: Casey Lee <cplee@nektos.com>
(cherry picked from commit e42a534b2e65d28b41532f9a44d18c0354aecfc2)

<!--start release-notes-assistant-->
<!--URL:https://code.forgejo.org/forgejo/runner-->
- other
  - [PR](https://code.forgejo.org/forgejo/runner/pulls/822): <!--number 822 --><!--line 0 --><!--description Y2hvcmU6ICJ1bmFibGUgdG8gZ2V0IGdpdCByZXBvIiBpcyBhIGRlYnVnIG1lc3NhZ2UsIG5vdCBhIHdhcm5pbmcgW3NraXAgY2FzY2FkZV0=-->chore: "unable to get git repo" is a debug message, not a warning [skip cascade]<!--description-->
<!--end release-notes-assistant-->

Co-authored-by: Steven Edwards <cureadvocate@gmail.com>
Reviewed-on: https://code.forgejo.org/forgejo/runner/pulls/822
Reviewed-by: Gusted <gusted@noreply.code.forgejo.org>
Co-authored-by: Earl Warren <contact@earl-warren.org>
Co-committed-by: Earl Warren <contact@earl-warren.org>
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.

act shouldn't complain when there's no remote repo

3 participants