Skip to content

Removed validator-ajv6 #4502

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

Merged
merged 3 commits into from
Feb 20, 2025
Merged

Removed validator-ajv6 #4502

merged 3 commits into from
Feb 20, 2025

Conversation

heath-freenome
Copy link
Member

Reasons for making this change

  • Removed all traces of the validator-ajv6 from everything except for CHANGELOG.md and v5 migration guide
  • Deleted the packages/validator-ajv
  • Updated all files that referenced that validator to remove them, updating documenation as needed

Checklist

  • I'm updating documentation
  • I'm adding or updating code
    • I've added and/or updated tests. I've run npx nx run-many --target=build --exclude=@rjsf/docs && npm run test:update to update snapshots, if needed.
    • I've updated docs if needed
    • I've updated the changelog with a description of the PR
  • I'm adding a new feature
    • I've updated the playground with an example use of the feature

- Removed all traces of the `validator-ajv6` from everything except for `CHANGELOG.md` and v5 migration guide
- Deleted the `packages/validator-ajv`
- Updated all files that referenced that validator to remove them, updating documenation as needed
Copy link
Contributor

@nickgros nickgros left a comment

Choose a reason for hiding this comment

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

One small change to revert, and also needs to be added to v6 changelog

@heath-freenome heath-freenome merged commit bdf1a2e into rjsf-v6 Feb 20, 2025
4 checks passed
@heath-freenome heath-freenome deleted the remove-validator-ajv6 branch February 20, 2025 18:39
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.

2 participants