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

Changes to sed command to use pr 107 of launcher #3816

Merged
merged 1 commit into from
May 1, 2024

Conversation

1000TurquoisePogs
Copy link
Member

PR zowe/launcher#107 which is primarily for the JCL PR, can be merged prior to it as long as this PR is merged which makes the current work compatible with it.

The incompatibility fixed here is just that the DISP=SHR argument is on a different line, so the sed command needs adjustment to that.

Copy link

github-actions bot commented May 1, 2024

build 5150 CANCELLED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8907990854

Copy link
Contributor

@JoeNemo JoeNemo left a comment

Choose a reason for hiding this comment

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

Approved.

@1000TurquoisePogs
Copy link
Member Author

The test at #3817 passed so I will merge this instead of running the same test twice.

@1000TurquoisePogs 1000TurquoisePogs merged commit 2b08d66 into v2.x/staging May 1, 2024
5 of 7 checks passed
@1000TurquoisePogs 1000TurquoisePogs deleted the feature/v2/use-pr-107 branch May 1, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants