Fix for mistake in indent. #2377
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
helene-nguyen
approved these changes
Jul 22, 2024
|
@piolug93 Thank you for your contribution, I've merged your changes but I need to re-indent because this return needs to be in this loop, a new PR has been made for this. Could you open an issue for this and detail the expected result? I've added some changes to have more information in logs in case the connector have an exception outside the main process. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The return was in a loop, when the indicator was composed, there was an exit too early. This resulted in the failure to create two indices for the filename|md5 type, for example.