Re: [Tarantool-patches] [PATCH luajit 8/8] ci: merge Linux and macOS workflows

[Sergey Kaplun via Tarantool-patches <tarantool-patches@dev.tarantool.org>]

Hi, Igor!

Thanks for the patch!
LGTM!

On 11.08.22, Igor Munkin wrote:
> As a result of two previous commits, there is no difference left between
> Linux and macOS workflows except the environment setup (environment
> variables, dependencies installation). Hence there is no reason to have a
> separate workflow file for each OS type.
> 
> Signed-off-by: Igor Munkin <imun@tarantool.org>
> ---
>  .github/actions/environment/action.yml        |  18 ---
>  .github/actions/setup-linux/README.md         |  12 ++
>  .github/actions/setup-linux/action.yml        |  19 +++
>  .github/actions/setup-macos/README.md         |  12 ++
>  .github/actions/setup-macos/action.yml        |  29 ++++
>  .../actions/{environment => setup}/README.md  |   5 +-
>  .github/actions/setup/action.yml              |  10 ++
>  .github/workflows/lint.yml                    |   8 +-
>  .github/workflows/linux-x86_64-ninja.yml      |   6 +-
>  .github/workflows/macos.yml                   | 142 ------------------
>  .github/workflows/{linux.yml => testing.yml}  | 113 +++++++++++---
>  11 files changed, 186 insertions(+), 188 deletions(-)
>  delete mode 100644 .github/actions/environment/action.yml
>  create mode 100644 .github/actions/setup-linux/README.md
>  create mode 100644 .github/actions/setup-linux/action.yml
>  create mode 100644 .github/actions/setup-macos/README.md
>  create mode 100644 .github/actions/setup-macos/action.yml
>  rename .github/actions/{environment => setup}/README.md (72%)
>  create mode 100644 .github/actions/setup/action.yml
>  delete mode 100644 .github/workflows/macos.yml
>  rename .github/workflows/{linux.yml => testing.yml} (52%)
> 

<snipped>

> -- 
> 2.34.0
> 

-- 
Best regards,
Sergey Kaplun