Skip to content

Update workflow.yml to support 2 most recent major versions #1417

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nodivbyzero
Copy link
Contributor

Fixes Or Enhances

Based on the https://github.com/go-playground/validator?tab=readme-ov-file#maintenance-and-support-for-sdk-major-versions

Make sure that you've checked the boxes below before you submit PR:

  • Tests exist or have been written that cover this particular change.

@go-playground/validator-maintainers

@nodivbyzero nodivbyzero requested a review from a team as a code owner April 16, 2025 19:01
@coveralls
Copy link

coveralls commented Apr 16, 2025

Coverage Status

coverage: 73.653%. remained the same
when pulling de8dffe on nodivbyzero:github-action-support-2-major-go-versions
into 8032f40 on go-playground:master.

@nodivbyzero
Copy link
Contributor Author

@deankarn @zemzale Can you please review this PR?
This change is required to address high-severity vulnerabilities in golang.org/x/crypto.

@deankarn
Copy link
Contributor

We only officially support the latest two versions, but is there any reason to not test the older version as well until if starts failing?

@nodivbyzero
Copy link
Contributor Author

@deankarn I'm just trying to understand the reasoning - why test the older version if it's no longer officially supported?
Are we planning to address any breaking issues if tests start failing on that version?

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.

4 participants