Skip to content
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

[v3] Dir to uss encoding flag #2350

Merged
merged 15 commits into from
Nov 8, 2024
Merged

[v3] Dir to uss encoding flag #2350

merged 15 commits into from
Nov 8, 2024

Conversation

jace-roell
Copy link
Contributor

@jace-roell jace-roell commented Nov 4, 2024

What It Does
Added support for --encoding flag to zowe upload dir-to-uss to allow for encoding uploaded directories for command group consistency

Review Checklist
I certify that I have:

Signed-off-by: jace-roell <[email protected]>
Signed-off-by: jace-roell <[email protected]>
@jace-roell jace-roell self-assigned this Nov 4, 2024
Copy link

codecov bot commented Nov 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.17%. Comparing base (f532f09) to head (ddf4203).
Report is 16 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2350   +/-   ##
=======================================
  Coverage   91.17%   91.17%           
=======================================
  Files         636      636           
  Lines       18052    18052           
  Branches     3779     3779           
=======================================
  Hits        16459    16459           
  Misses       1592     1592           
  Partials        1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

jace-roell and others added 2 commits November 4, 2024 16:15
@jace-roell jace-roell marked this pull request as ready for review November 5, 2024 16:49
@jace-roell jace-roell changed the title Dir to uss encoding flag [v3] Dir to uss encoding flag Nov 5, 2024
Copy link
Member

@t1m0thyj t1m0thyj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @jace-roell for the quick fix and for adding a system test!

Copy link
Member

@traeok traeok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for adding the encoding flag! 😋

packages/cli/CHANGELOG.md Outdated Show resolved Hide resolved
packages/zosfiles/CHANGELOG.md Outdated Show resolved Hide resolved
Signed-off-by: jace-roell <[email protected]>
Copy link
Contributor

@anaxceron anaxceron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting minor edits to the changelog entries

packages/cli/CHANGELOG.md Outdated Show resolved Hide resolved
packages/zosfiles/CHANGELOG.md Outdated Show resolved Hide resolved
Signed-off-by: jace-roell <[email protected]>
Signed-off-by: jace-roell <[email protected]>
jace-roell and others added 2 commits November 7, 2024 12:48
Signed-off-by: jace-roell <[email protected]>
Copy link
Member

@zFernand0 zFernand0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM as-is!
but left a couple of comments with some minor changes 😋

packages/zosfiles/CHANGELOG.md Show resolved Hide resolved
packages/zosfiles/src/methods/upload/Upload.ts Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Nov 8, 2024

@awharn awharn merged commit 3004122 into master Nov 8, 2024
19 checks passed
@awharn awharn deleted the dir-to-uss-encoding branch November 8, 2024 19:07
@awharn awharn restored the dir-to-uss-encoding branch November 8, 2024 19:08
@awharn awharn deleted the dir-to-uss-encoding branch November 8, 2024 19:08
@awharn awharn added the release-current Indicates that there is no new functionality being delivered label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-current Indicates that there is no new functionality being delivered
Projects
Status: Closed
Development

Successfully merging this pull request may close these issues.

zos-files upload dir-to-uss doesn't support all z/OSMF profile options
8 participants