-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Zos download attributes #2335
base: master
Are you sure you want to change the base?
Zos download attributes #2335
Conversation
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2335 +/- ##
==========================================
+ Coverage 91.15% 91.16% +0.01%
==========================================
Files 636 636
Lines 18029 18034 +5
Branches 3767 3877 +110
==========================================
+ Hits 16434 16441 +7
+ Misses 1594 1592 -2
Partials 1 1 ☔ View full report in Codecov by Sentry. |
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
packages/zosfiles/__tests__/__system__/methods/download/Download.system.test.ts
Fixed
Show fixed
Hide fixed
Signed-off-by: jace-roell <jace.roell@hotmail.com>
packages/zosfiles/__tests__/__system__/methods/download/Download.system.test.ts
Fixed
Show fixed
Hide fixed
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: Jace Roell <111985297+jace-roell@users.noreply.github.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty good, thanks @jace-roell! Left a few comments
Co-authored-by: Timothy Johnson <timothy.johnson@broadcom.com> Signed-off-by: Jace Roell <111985297+jace-roell@users.noreply.github.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple recommended changes and a question.
packages/zosfiles/__tests__/__system__/methods/download/__resources__/download_file.sh
Outdated
Show resolved
Hide resolved
packages/zosfiles/__tests__/__system__/methods/download/__resources__/upload_file_to_uss.sh
Outdated
Show resolved
Hide resolved
...es/zosfiles/__tests__/__system__/methods/download/__resources__/upload_file_to_uss_binary.sh
Outdated
Show resolved
Hide resolved
Signed-off-by: jace-roell <jace.roell@hotmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made some edit suggestions.
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Signed-off-by: jace-roell <jace.roell@hotmail.com>
Quality Gate passedIssues Measures |
What It Does
Added optional
--attributes
flag tozowe zos-files download uss-file
to allow passing a .zosattributes file path for download encoding format.Review Checklist
I certify that I have: