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

Cics READ improvements #2576

Open
wants to merge 12 commits into
base: development
Choose a base branch
from

Conversation

mm-broadcom
Copy link
Contributor

How Has This Been Tested?

Wrote some tests for the fullest and minimal extent allowed for READ along with invalid test case(s).

Checklist:

  • Each of my commits contains one meaningful change
  • I have performed rebase of my branch on top of the development
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have checked my code and corrected any misspellings

@mm-broadcom mm-broadcom marked this pull request as draft November 14, 2024 17:36
@mm-broadcom
Copy link
Contributor Author

Moving to draft. Is reliant upon functions from the WEB fix PR #2469. (Was developed and tested off of that branch before moving to its own one.)

@mm-broadcom mm-broadcom force-pushed the CICS-READ-Improvements branch from 9fd507b to 4093cb2 Compare November 15, 2024 15:49
@mm-broadcom mm-broadcom marked this pull request as ready for review November 18, 2024 13:08
Added support for the obsolete DATASET keyword. When both are used, inform the user to use FILE instead of DATASET. (They are synonymous to one another, but FILE is the one to use going forward.)
@mm-broadcom
Copy link
Contributor Author

@KutluOzel-b Could you review this?

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.

4 participants