You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
This version of the code separates the different responsibilities into separate functions, and uses the log package instead of os.Exit() and panic() to handle errors. Additionally, it uses the ioutil package instead of os package to read the file, which simplify the code and it also returns errors instead of panicking on them, making it easier to handle them at a higher level in the application.
This version of the code separates the different responsibilities into separate functions, and uses the log package instead of `os.Exit()` and `panic()` to handle errors.
Additionally, it uses the ioutil package instead of os package to read the file, which simplify the code and it also returns errors instead of panicking on them, making it easier to handle them at a higher level in the application.
Merging #5658 (bec0d76) into master (50b01d1) will decrease coverage by 1%.
The diff coverage is n/a.
❗ Current head bec0d76 differs from pull request most recent head dce7f12. Consider uploading reports for the commit dce7f12 to get more accurate results
hi, @bhaskarvilles, sorry there is no reward for single behavior, but do you want to become a long-term contributor to the Venus community and get some rewards?
hi, @bhaskarvilles, sorry there is no reward for single behavior, but do you want to become a long-term contributor to the Venus community and get some rewards?
Hey @Joss-Hua , can you give me more details please?
hi, @bhaskarvilles, sorry there is no reward for single behavior, but do you want to become a long-term contributor to the Venus community and get some rewards?
Hey @Joss-Hua , can you give me more details please?
What is your filecoin slack handle? I'll contact you.
hi, @bhaskarvilles, sorry there is no reward for single behavior, but do you want to become a long-term contributor to the Venus community and get some rewards?
Hey @Joss-Hua , can you give me more details please?
What is your filecoin slack handle? I'll contact you.
My handle is bhaskarvilles, I'm also in the venus fil channel also
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.
This version of the code separates the different responsibilities into separate functions, and uses the log package instead of
os.Exit()andpanic()to handle errors. Additionally, it uses the ioutil package instead of os package to read the file, which simplify the code and it also returns errors instead of panicking on them, making it easier to handle them at a higher level in the application.