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

Update COBOL Programming Course #4 - Testing.md #331

Merged
merged 1 commit into from
Mar 20, 2025

Conversation

JulioVS
Copy link
Contributor

@JulioVS JulioVS commented Mar 15, 2025

Fixes and updates I made while doing these labs in March 2025.-

Proposed changes

I made updates to the labs in order to be able to more easily follow these practices.

Fixes # (issue)

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)

Fixes and updates I made while doing these labs in March 2025.-

Signed-off-by: Julio Errecart <120730394+JulioVS@users.noreply.github.com>
@MikeBauerCA
Copy link
Contributor

Hello @JulioVS, thank you for raising a PR! We will review shortly.

Copy link
Contributor

@alim395 alim395 left a comment

Choose a reason for hiding this comment

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

Everything appears to be in order. Looking forward to the new update.

@MikeBauerCA
Copy link
Contributor

Thanks @alim395. I'll approve and merge. Very nice contribution @JulioVS - thank you! 🚀

@MikeBauerCA MikeBauerCA merged commit e7dae24 into openmainframeproject:master Mar 20, 2025
1 check passed
@JulioVS
Copy link
Contributor Author

JulioVS commented Mar 21, 2025

My pleasure.
All the best,
Julio.-
(Back to the wonderful Mainframe after a 25-year “lost weekend”:)

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.

3 participants