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

Update workflows #2410

Merged
merged 2 commits into from May 5, 2024
Merged

Conversation

rotu
Copy link
Contributor

@rotu rotu commented Apr 6, 2024

What issue does this pull request resolve?

Silence warnings about outdated actions e.g. https://github.com/ajv-validator/ajv/actions/runs/8490491525
Switch to ${{}} syntax in if statement for quoting expressions (necessary when condition starts with !)

What changes did you make?

Is there anything that requires more attention while reviewing?

@jasoniangreen
Copy link
Collaborator

This one looks good to me @epoberezkin, it passes the build with the updates to the build script so should be good.

@jasoniangreen
Copy link
Collaborator

@epoberezkin I am going to merge this as it is something I was planning to do anyway and in the worse case that it breaks, it can only break workflows which will only impact me.

@jasoniangreen jasoniangreen merged commit d9bd01b into ajv-validator:master May 5, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants