-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fixed SonarCloud task void issues related #664
base: development
Are you sure you want to change the base?
Fixed SonarCloud task void issues related #664
Conversation
Kudos, SonarCloud Quality Gate passed! |
@@ -270,7 +290,7 @@ private void Execute(ISession session, UserRuleVerification userRuleVerification | |||
/// <param name="ruleVerification"> | |||
/// The <see cref="RuleVerification"/> that is to be updated. | |||
/// </param> | |||
private async void UpdateExecutedOn(ISession session, RuleVerification ruleVerification) | |||
private void UpdateExecutedOn(ISession session, RuleVerification ruleVerification) |
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.
probably best to make this private async Task ...
/// <summary> | ||
/// The Logger. | ||
/// </summary> | ||
private static Logger logger = LogManager.GetCurrentClassLogger(); |
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.
better to add more logging instead of removing the logger
@@ -262,13 +237,14 @@ private async void ExecuteOk() | |||
var dal = this.dals.Single(x => x.Metadata.DalType == DalType.File); | |||
var dalInstance = (IDal)Activator.CreateInstance(dal.Value.GetType()); | |||
|
|||
var fileExportSession = new Session(dalInstance, creds); | |||
_ = new Session(dalInstance, creds); |
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.
not happy about the _
does not make it more readible, please revert
@@ -494,7 +478,7 @@ private async void CreateBinaryRelationship(DiagramConnector connector) | |||
try | |||
{ | |||
var operationContainer = containerTransaction.FinalizeTransaction(); | |||
await this.Session.Write(operationContainer); | |||
this.Session.Write(operationContainer).GetAwaiter().GetResult(); |
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.
please revert
Prerequisites
Description