Skip to content

fix invalid definition in plugins-ci and plugins-ci-redis#84

Merged
mcollina merged 2 commits intofastify:mainfrom
rluvaton:patch-1
Jun 9, 2023
Merged

fix invalid definition in plugins-ci and plugins-ci-redis#84
mcollina merged 2 commits intofastify:mainfrom
rluvaton:patch-1

Conversation

@rluvaton
Copy link
Copy Markdown
Contributor

@rluvaton rluvaton commented Jun 9, 2023 •
edited
Loading

fix:

Example of usage:

strategy:
  matrix:
    os: [macos-latest, windows-latest]
    version: [12, 14, 16]
    environment: [staging, production]
    exclude:
      - os: macos-latest
        version: 12
        environment: production
      - os: windows-latest
        version: 16
runs-on: ${{ matrix.os }}

from Github Actions docs - Using a matrix for your jobs - Excluding matrix configurations

@rluvaton rluvaton changed the title fix #8 fix invalid definition in plugins-ci and plugins-ci-redis Jun 9, 2023
@rluvaton
Copy link
Copy Markdown
Contributor Author

rluvaton commented Jun 9, 2023

@mcollina

@rluvaton rluvaton mentioned this pull request Jun 9, 2023
Closed
mcollina
mcollina approved these changes Jun 9, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina merged commit 3135dc1 into fastify:main Jun 9, 2023
@rluvaton rluvaton deleted the patch-1 branch June 9, 2023 13:46
@optic-release-automation optic-release-automation bot mentioned this pull request Jun 9, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mcollina mcollina mcollina approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rluvaton @mcollina