Skip to content

Comments

Implement suggested changes#5

Merged
MohamedSabthar merged 3 commits intoballerina-platform:mainfrom
CharanaManawathilake:main
Jan 17, 2025
Merged

Implement suggested changes#5
MohamedSabthar merged 3 commits intoballerina-platform:mainfrom
CharanaManawathilake:main

Conversation

@CharanaManawathilake
Copy link
Contributor

Purpose

Examples

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests
  • Updated the spec
  • Checked native-image compatibility

Comment on lines 87 to 92
if cancelResponse.status == "COMPLETE" {
io:println("The import was cancelled successfully");
}
else {
io:println("The import cancellation is in the state : " + cancelResponse.status);
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if cancelResponse.status == "COMPLETE" {
io:println("The import was cancelled successfully");
}
else {
io:println("The import cancellation is in the state : " + cancelResponse.status);
}
if cancelResponse.status == "COMPLETE" {
io:println("The import was cancelled successfully");
return;
}
io:println("The import cancellation is in the state : " + cancelResponse.status);

else {
io:println("The import cancellation is in the state : " + cancelResponse.status);
}
return io:println("Failed to cancel the import");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
return io:println("Failed to cancel the import");
io:println("Failed to cancel the import");
return;

@MohamedSabthar MohamedSabthar merged commit 3464908 into ballerina-platform:main Jan 17, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants