-
Notifications
You must be signed in to change notification settings - Fork 61
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
Enhancement/lite db upgrade #3389
Changes from 18 commits
7f73abd
31abb5e
67bb0e3
2c87245
29fb70b
fdc7854
a9ef1c7
6e31aad
9294d59
f500c16
0be1f94
a32e6f6
d97e5fc
25346f3
c27a2c7
c77f167
ea60a25
815b597
c7f643b
aaac8bd
e1566e9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -34,7 +34,7 @@ | |||||||
|
||||||||
<ItemGroup> | ||||||||
<PackageReference Include="Ginger.External" Version="1.0.0" /> | ||||||||
<PackageReference Include="LiteDB" Version="4.1.4" /> | ||||||||
<PackageReference Include="LiteDB" Version="5.0.17" /> | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The HintPath for LiteDB.dll is still pointing to the old version. This should be updated or removed if it's no longer necessary due to the package reference. - <HintPath>..\..\..\..\..\.nuget\packages\litedb\4.1.4\lib\net40\LiteDB.dll</HintPath>
+ <HintPath>..\..\..\..\..\.nuget\packages\litedb\5.0.17\lib\netstandard2.0\LiteDB.dll</HintPath> Committable suggestion
Suggested change
|
||||||||
<PackageReference Include="Microsoft.CodeAnalysis" Version="3.7.0" /> | ||||||||
<PackageReference Include="Microsoft.CodeAnalysis.Common" Version="3.7.0" /> | ||||||||
<PackageReference Include="Microsoft.CodeAnalysis.CSharp" Version="3.7.0" /> | ||||||||
|
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.
The catch block in the
GenerateBFReport
method is empty. It's recommended to at least log the exception to aid in debugging if an error occurs.try { // ... existing code ... } catch (Exception ex) { + Reporter.ToLog(eLogLevel.ERROR, "Failed to generate business flow report", ex); }
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.
ok, will make the necessary changes
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.
@manas-droid Great, glad to hear that you'll make the changes. If you need any more assistance, feel free to ask!
🎉
New Learnings Added
If you found this review helpful, would you consider giving us a shout-out on X?
Thank you for using CodeRabbit!