Code style fix for precip variabliity codes #822
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
Thank you for modifying the code style. I have checked the analysis result with the modified code and found that the modification does not influence the analysis result. |
msahn
approved these changes
Nov 24, 2021
…lity_ljw_for_pre-commit
|
@msahn Excellent, thank you for checking it! |
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.
@msahn in this PR, I've separated code style change for your branch. Code style checkup and revisions were done by
pre-commt(as documented here), which now required for Github Action CI process.Can you please check to see if any influence to the result? I suspect not, because all changes in this PR to your branch is only for code style, but just in case it would be safer to check result consistency before merging this to your branch.