-
Notifications
You must be signed in to change notification settings - Fork 70
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
Proccess file sync #12246
Proccess file sync #12246
Conversation
25ada23
to
29630a1
Compare
29630a1
to
1bd7b07
Compare
...esigner.Tests/Controllers/ProcessModelingController/UpsertProcessDefinitionAndNotifyTests.cs
Fixed
Show fixed
Hide fixed
...esigner.Tests/Controllers/ProcessModelingController/UpsertProcessDefinitionAndNotifyTests.cs
Fixed
Show fixed
Hide fixed
4ef02fb
to
db7fc44
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #12246 +/- ##
=======================================
Coverage 86.42% 86.42%
=======================================
Files 1188 1188
Lines 18049 18049
Branches 2292 2292
=======================================
Hits 15598 15598
Misses 2163 2163
Partials 288 288 ☔ View full report in Codecov by Sentry. |
db7fc44
to
d919067
Compare
d919067
to
ce0ff2b
Compare
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.
Looks great!
I added one comment, just for consideration/out of curiosity 😊
Description
Related Issue(s)
Verification
Documentation