-
Notifications
You must be signed in to change notification settings - Fork 44
feat(ama-openapi): add dynamic schema #3778
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
Conversation
414f8e6 to
14878e3
Compare
|
View your CI Pipeline Execution ↗ for commit 6b17da9
☁️ Nx Cloud last updated this comment at |
1ac7672 to
ee750fd
Compare
ee750fd to
c9ce10f
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
c9ce10f to
ba6f635
Compare
ba6f635 to
71ec6c3
Compare
71ec6c3 to
772b76f
Compare
772b76f to
e613afc
Compare
e613afc to
c7f987a
Compare
d35b6b9 to
218a83e
Compare
218a83e to
6b17da9
Compare
| ignoredPackages: [ | ||
| '@o3r/build-helpers', | ||
| '@o3r/workspace-helpers' | ||
| ignoredDependencies: [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it really linked to this PR ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same comment for the shared config
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to do this bugfix because of the version of globby which are not the same between js and mjs package and the linter was falsely failing.
I do not think I need to move this fix out to block this pr.
Proposed change
Add dynamic schema generator to help the writing of configuration
Related issues
- No issue associated -