Skip to content

Conversation

MikeBauerCA
Copy link
Contributor

@MikeBauerCA MikeBauerCA commented Dec 18, 2024

Proposed changes

This PR removes the "Installing Prerequisites" section from the installation of VS Code and extensions chapter. Node.js is only a prerequisite for Zowe CLI so that content has been moved to the Zowe CLI installation section. The Java SDK was only required for Z Open Editor so the content now refers the reader to the prerequisite information shown on the VS Code marketplace for that extension at https://marketplace.visualstudio.com/items?itemName=IBM.zopeneditor#prerequisites

Also in this PR is a small change to the Front_Matter.tex files to render images appropriately in the generated PDFs.

Type of change

What type of changes does your PR introduce to the COBOL Programming Course? Put an x in the boxes that apply.

  • Bug fix (change which fixes one or more issues)
  • New feature (change which adds functionality or features to the course)
  • Translations (change which adds or modifies translations of the course)
  • Documentation (change which modifies documentation related to the course)
  • This change requires an update to the course's z/OS environment

Checklist:

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This checklist will be used as reference for both the contributor and the reviewer

  • I have read the Contributing Guideline
  • I have included a title and description for this PR
  • I have DCO-signed all of my commits that are included in this PR
  • I have tested it manually and there are no regressions found
  • I have commented my code, particularly in hard-to-understand areas (if appropriate)
  • I have made corresponding changes to the documentation (if appropriate)

Signed-off-by: Michael Bauer <michael.bauer2@broadcom.com>
Signed-off-by: Michael Bauer <michael.bauer2@broadcom.com>
@MikeBauerCA MikeBauerCA requested a review from tanto259 December 18, 2024 21:59
Signed-off-by: Michael Bauer <michael.bauer2@broadcom.com>
Signed-off-by: Michael Bauer <michael.bauer2@broadcom.com>
Copy link
Member

@tanto259 tanto259 left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks, Mike!

@MikeBauerCA
Copy link
Contributor Author

Thank you, I'm going to proceed with the merge. For any additional changes prior to the upcoming release, we can create a new branch.

@MikeBauerCA MikeBauerCA merged commit 12b84c5 into master Dec 19, 2024
1 check passed
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