fix docker executor on windows and local actions #34

Merged
ChristopherHX merged 1 commits from refs/pull/34/head into main 2023-03-06 05:24:32 +00:00
ChristopherHX commented 2023-03-03 22:55:52 +00:00 (Migrated from gitea.com)

If the Workdir field doesn't ends with the filepath seperator,
bad things happen

Fixes #33

Sample for host mode on windows, needs be adjusted for linux e.g. replace pwsh with bash
Also fixes

on: push
jobs:
  _:
    runs-on: self-hosted
    steps:
    - uses: actions/checkout@v3
      with:
        path: subdir/action
    - uses: ./subdir/action

with an action.yml in the same repo

runs:
  using: composite
  steps:
    - run: |
        echo "Hello World"
      shell: pwsh
If the Workdir field doesn't ends with the filepath seperator, bad things happen Fixes #33 Sample for host mode on windows, needs be adjusted for linux e.g. replace pwsh with bash Also fixes ```yaml on: push jobs: _: runs-on: self-hosted steps: - uses: actions/checkout@v3 with: path: subdir/action - uses: ./subdir/action ``` with an action.yml in the same repo ```yaml runs: using: composite steps: - run: | echo "Hello World" shell: pwsh ```
lunny commented 2023-03-04 11:20:03 +00:00 (Migrated from gitea.com)

I want to know why the CI hasn't been triggered.

I want to know why the CI hasn't been triggered.
a1012112796 commented 2023-03-04 11:29:23 +00:00 (Migrated from gitea.com)

I want to know why the CI hasn't been triggered.

ci has been triggered but no commit status was created.

https://gitea.com/gitea/act_runner/actions/runs/74

> I want to know why the CI hasn't been triggered. ci has been triggered but no commit status was created. https://gitea.com/gitea/act_runner/actions/runs/74
wolfogre (Migrated from gitea.com) approved these changes 2023-03-06 02:00:36 +00:00
wolfogre commented 2023-03-06 02:01:13 +00:00 (Migrated from gitea.com)

@Zettat123 Could you please help do some tests and review?

@Zettat123 Could you please help do some tests and review?
Zettat123 commented 2023-03-06 04:03:51 +00:00 (Migrated from gitea.com)

@Zettat123 Could you please help do some tests and review?

This change works well in my test. So LGTM.

> @Zettat123 Could you please help do some tests and review? This change works well in my test. So LGTM.
lunny (Migrated from gitea.com) approved these changes 2023-03-06 05:22:28 +00:00
Sign in to join this conversation.