Skip to content

API schema processors #890

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 4 commits into from
Apr 14, 2025
Merged

API schema processors #890

merged 4 commits into from
Apr 14, 2025

Conversation

dan-lee
Copy link
Contributor

@dan-lee dan-lee commented Apr 11, 2025

@dan-lee dan-lee added the feature Feature label Apr 11, 2025
Copy link

vercel bot commented Apr 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
zudoku-cosmo-cargo ✅ Ready (Inspect) Visit Preview Apr 11, 2025 3:17pm
zudoku-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 11, 2025 3:17pm
zudoku-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 11, 2025 3:17pm

Copy link

nx-cloud bot commented Apr 11, 2025

View your CI Pipeline Execution ↗ for commit 1c66c54.

Command Status Duration Result
nx run-many -t build --projects=tag:example ✅ Succeeded 1m 12s View ↗
nx run-many -t lint:ci format:ci build:ci test ✅ Succeeded 25s View ↗

☁️ Nx Cloud last updated this comment at 2025-04-11 15:18:44 UTC

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 22 out of 26 changed files in this pull request and generated no comments.

Files not reviewed (4)
  • docs/pages/configuration/build-configuration.mdx: Language not supported
  • docs/pages/guides/processors.mdx: Language not supported
  • examples/cosmo-cargo/tsconfig.json: Language not supported
  • packages/zudoku/package.json: Language not supported
Comments suppressed due to low confidence (1)

packages/zudoku/src/config/validators/common.ts:80

  • The removal of the 'postProcessors' merge from the API schema is a breaking change if existing configurations rely on it. Please update the relevant documentation or migration guides to assist users in transitioning to the new build processor configuration.
    .merge(ApiConfigSchema),

@dan-lee dan-lee merged commit 466c776 into main Apr 14, 2025
8 checks passed
@dan-lee dan-lee deleted the feat/processors branch April 14, 2025 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature
Development

Successfully merging this pull request may close these issues.

2 participants