-
Notifications
You must be signed in to change notification settings - Fork 72
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
Fix OpenAPI TypeScript types #5437
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
fides Run #10689
Run Properties:
|
Project |
fides
|
Branch Review |
refs/pull/5437/merge
|
Run status |
Passed #10689
|
Run duration | 00m 37s |
Commit |
a88727a992 ℹ️: Merge 56cbbeed278c4edd220bf1d2d6da4d16d70d3f9d into eac2542b07b76ebc9a8a15386a1f...
|
Committer | jpople |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
4
|
Upgrade your plan to view test results. | |
View all changes introduced in this branch ↗︎ |
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'm assuming this is mostly automated updates. Anything you think I should test or is this more of a "general regression test" kind of thing?
@gilluminate General regression, the only real manual change aside from adding some fallback |
fides Run #10720
Run Properties:
|
Project |
fides
|
Branch Review |
main
|
Run status |
Passed #10720
|
Run duration | 00m 40s |
Commit |
008cf6818d: Fix OpenAPI TypeScript types (#5437)
|
Committer | jpople |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
4
|
Upgrade your plan to view test results. | |
View all changes introduced in this branch ↗︎ |
Closes HJ-17
Code Changes
Pre-Merge Checklist
CHANGELOG.md