Skip to content

Conversation

borderite
Copy link
Contributor

  • While the test assumes that WS-ARRAY is a string, the template cobol code delcared it as an integer array. The revised template now declares WS-ARRAY as a string.

- While the test assumes that WS-ARRAY is a string, the template cobol
code delcared it as an integer array. The revised template now
declares WS-ARRAY as a string.
Copy link

github-actions bot commented Aug 9, 2025

Hello. Thanks for opening a PR on Exercism 🙂

We ask that all changes to Exercism are discussed on our Community Forum before being opened on GitHub. To enforce this, we automatically close all PRs that are submitted. That doesn't mean your PR is rejected but that we want the initial discussion about it to happen on our forum where a wide range of key contributors across the Exercism ecosystem can weigh in.

You can use this link to copy this into a new topic on the forum. If we decide the PR is appropriate, we'll reopen it and continue with it, so please don't delete your local branch.

If you're interested in learning more about this auto-responder, please read this blog post.


Note: If this PR has been pre-approved, please link back to this PR on the forum thread and a maintainer or staff member will reopen it.

@github-actions github-actions bot closed this Aug 9, 2025
@axtens axtens reopened this Aug 11, 2025
@axtens
Copy link
Member

axtens commented Aug 11, 2025

IIRC (and there's some doubt) I may have put

       01 WS-ARRAY-TABLE.
         02 WS-ARRAY               OCCURS 1 TO 20 DEPENDING ON WS-COUNT.
            05 ROWELEM             PIC 9(4).

in as a clue to the student. Maybe that's a bit too much hand-holding.

@axtens axtens requested a review from kapitaali August 11, 2025 01:38
@axtens
Copy link
Member

axtens commented Aug 11, 2025

@kapitaali Comments?

@kapitaali
Copy link
Contributor

kapitaali commented Aug 11, 2025

Aye ok yes, fine by me.

@axtens axtens requested review from axtens and removed request for kapitaali August 12, 2025 06:39
@axtens axtens merged commit 99a20ae into exercism:main Aug 12, 2025
6 checks 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.

3 participants