Closed willief closed 1 month ago
Thanks for this @willief . For simplicity i've made https://github.com/maidsafe/safe_network/pull/1893 which adds the title you have (almost perfect commit convention) to the commit msg.
That should now pass the CI checks therein
(note the lack of sentence case and no space before subject, ie fix(cli)
not fix (cli)
)
Thanks Joshuef, there was more work in the commit message than in the issue itself :-)
On Sat, 15 Jun 2024 at 23:19, joshuef @.***> wrote:
Thanks for this @willief https://github.com/willief . For simplicity i've made #1893 https://github.com/maidsafe/safe_network/pull/1893 which adds the title you have (almost perfect commit convention) to the commit msg.
That should now pass the CI checks therein
(note the lack of sentence case and no space before subject, ie fix(cli) not fix (cli))
— Reply to this email directly, view it on GitHub https://github.com/maidsafe/safe_network/pull/1892#issuecomment-2170944598, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACHW3IKQICDUPH5CJKIBPTZHS4YXAVCNFSM6AAAAABJKTE3KWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZQHE2DINJZHA . You are receiving this because you were mentioned.Message ID: @.***>
-- Regards Willie Fleming
fix: Correct a misleading error msg in sn_cli/src/bin/subcommands/files.rs
Description
Existing error message referred to "files upload" This changes the error to refer to "files download"
Related Issue
Fixes #1890
Type of Change
Please mark the types of changes made in this pull request.
Checklist
Please ensure all of the following tasks have been completed: