Skip to content
This repository was archived by the owner on Mar 20, 2025. It is now read-only.

Conversation

danez
Copy link
Contributor

@danez danez commented Nov 18, 2022

Moves the validation from @netlify/build to this repo.

It is pretty much a copy, except:

  • added new tests
  • changed bundle format to be an enum and also is validated as such
  • uses better-ajv-errors for pretty printed errors (PR to remove emojis)
  • initializes ajv lazy

netlify/build#4710

@danez danez self-assigned this Nov 18, 2022
@danez danez requested a review from a team November 18, 2022 16:06
Copy link
Contributor

@Skn0tt Skn0tt left a comment

Choose a reason for hiding this comment

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

looks good to me 👍

@kodiakhq kodiakhq bot merged commit 17ee035 into main Nov 21, 2022
@kodiakhq kodiakhq bot deleted the validation branch November 21, 2022 10:07
Skn0tt pushed a commit to netlify/build that referenced this pull request Apr 23, 2024
* feat: add manifest validation

* chore: disable color in test
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants