[cpackget] refined --skip-touch option

[thorstendb-ARM]

added option also to other commands touching pack.idx

[codecov-commenter]

Codecov Report

Patch coverage: 55.55% and project coverage change: -0.07 ⚠️

Comparison is base (64c7ae5) 79.00% compared to head (bfa6b15) 78.93%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #190      +/-   ##
==========================================
- Coverage   79.00%   78.93%   -0.07%     
==========================================
  Files          22       22              
  Lines        2572     2583      +11     
==========================================
+ Hits         2032     2039       +7     
- Misses        399      403       +4     
  Partials      141      141              

Impacted Files	Coverage Δ
cmd/utils/utils.go	83.79% <0.00%> (-1.04%)	⬇️
cmd/installer/root.go	70.43% <20.00%> (ø)
cmd/commands/add.go	90.47% <100.00%> (+0.15%)	⬆️
cmd/commands/init.go	86.36% <100.00%> (+1.36%)	⬆️
cmd/commands/pack.go	86.48% <100.00%> (ø)
cmd/commands/rm.go	91.17% <100.00%> (+0.55%)	⬆️
cmd/commands/update_index.go	95.45% <100.00%> (+0.45%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[bgn42]

Looks good

[bgn42]

--skip-touch originated from PackUnzip where it had the same functionality and there also was no short option. Also, this is an internal option not used directly by a user. Von: Sourabh Mehta ***@***.***> Gesendet: Mittwoch, 28. Juni 2023 16:08 An: Open-CMSIS-Pack/cpackget ***@***.***> Cc: Bernd-Gunter Nitzler ***@***.***>; Review requested ***@***.***> Betreff: Re: [Open-CMSIS-Pack/cpackget] [cpackget] refined --skip-touch option (PR #190) @soumeh01 commented on this pull request.

________________________________ In cmd/commands/init.go<#190 (comment)>:

@@ -53,4 +57,5 @@ The index-url is mandatory. Ex "cpackget init --pack-root path/to/mypackroot htt

func init() { InitCmd.Flags().BoolVarP(&initCmdFlags.downloadPdscFiles, "all-pdsc-files", "a", false, "downloads all the latest .pdsc files from the public index") InitCmd.Flags().BoolVarP(&initCmdFlags.encodedProgress, "encoded-progress", "E", false, "Reports encoded progress for files and download when used by other tools") + InitCmd.Flags().BoolVar(&initCmdFlags.skipTouch, "skip-touch", false, "do not touch pack.idx") can you confirm if this needs only long options --skip-touch and no short option? — Reply to this email directly, view it on GitHub<#190 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AWCKMYDJYIL5TQRAGOITGCTXNQ3CXANCNFSM6AAAAAAZXE2ENY>. You are receiving this because your review was requested.Message ID: ***@***.******@***.***>> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.