Skip to content
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

ci: migrate to fastify reusable workflow #93

Merged
merged 1 commit into from
May 18, 2022
Merged

ci: migrate to fastify reusable workflow #93

merged 1 commit into from
May 18, 2022

Conversation

Fdawgs
Copy link
Member

@Fdawgs Fdawgs commented May 17, 2022

Checklist

@Fdawgs Fdawgs marked this pull request as draft May 17, 2022 09:36
@mcollina
Copy link
Member

Why is a draft?

@Fdawgs
Copy link
Member Author

Fdawgs commented May 18, 2022

Why is a draft?

There's something wrong with the Postgres and MySQL reusable workflow syntax, so set these to drafts until I can sort it.

@climba03003
Copy link
Member

climba03003 commented May 18, 2022

Why is a draft?

There's something wrong with the Postgres and MySQL reusable workflow syntax, so set these to drafts until I can sort it.

I think it is missing a quote for the string. fastify/workflows#26

The workflow is not valid. fastify/workflows/.github/workflows/plugins-ci-mysql.yml@v3: (Line: 68, Col: 14, Idx: 1601) - (Line: 68, Col: 19, Idx: 1606): While scanning a plain scalar, find unexpected ':'. Unexpected type '' encountered while reading 'root'. The type 'MappingToken' was expected.

@Fdawgs Fdawgs marked this pull request as ready for review May 18, 2022 06:36
@Fdawgs Fdawgs merged commit 55f7adc into master May 18, 2022
@Fdawgs Fdawgs deleted the ci/gh-actions branch May 18, 2022 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants