-
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
HJ-139 - Fix Pytest-Ctl-External tests #5457
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
fides Run #10850
Run Properties:
|
Project |
fides
|
Branch Review |
refs/pull/5457/merge
|
Run status |
Passed #10850
|
Run duration | 00m 39s |
Commit |
d9f280851e ℹ️: Merge c478d8a1491da8e3864c14e8c4ff30d2ff6cba68 into 722444864fa420685ce901fc6e90...
|
Committer | Andres Torres |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
4
|
View all changes introduced in this branch ↗︎ |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5457 +/- ##
=======================================
Coverage 85.42% 85.42%
=======================================
Files 385 385
Lines 24152 24152
Branches 2630 2630
=======================================
+ Hits 20631 20633 +2
+ Misses 2967 2966 -1
+ Partials 554 553 -1 ☔ View full report in Codecov by Sentry. |
env: | ||
OP_SERVICE_ACCOUNT_TOKEN: ${{ secrets.OP_SERVICE_ACCOUNT_TOKEN }} | ||
|
||
# Secrets to pull from 1Password |
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.
This is gonna be great for centralizing the privacy information!
fides Run #10852
Run Properties:
|
Project |
fides
|
Branch Review |
main
|
Run status |
Passed #10852
|
Run duration | 00m 37s |
Commit |
04461d0687: HJ-139 - Fix Pytest-Ctl-External tests (#5457)
|
Committer | Andres Torres |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
4
|
View all changes introduced in this branch ↗︎ |
Closes #HJ-139
Description Of Changes
Fix Pytest-Ctl-External tests
Code Changes
Steps to Confirm
Pre-Merge Checklist
CHANGELOG.md
main
downgrade()
migration is correct and works