Skip to content

Fix naming inconsistencies across directories and files#60

Open
devin-ai-integration[bot] wants to merge 1 commit intomainfrom
devin/1770850207-fix-naming-inconsistencies
Open

Fix naming inconsistencies across directories and files#60
devin-ai-integration[bot] wants to merge 1 commit intomainfrom
devin/1770850207-fix-naming-inconsistencies

Conversation

@devin-ai-integration
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot commented Feb 11, 2026

Fix directory and file naming inconsistencies across repo

Summary

Renames directories and files so that each directory's .cbl file matches the directory name, and standardizes on underscores (instead of hyphens) for consistency across the repo. All renames use git mv to preserve history.

Renames:

Before After
redifines/ redefines/ (typo fix)
accept/accept-secure.cbl accept/accept_secure.cbl
display_test/display-test.cbl display_test/display_test.cbl
merge_sort/merge_sort_test.cbl merge_sort/merge_sort.cbl
mouse/mouse_example.cbl mouse/mouse.cbl
report_writer/report_test.cbl report_writer/report_writer.cbl
screen_size/get_screen_size.cbl screen_size/screen_size.cbl

README updates: accept/README.md and screen_size/README.md updated to reference new filenames.

Review & Testing Checklist for Human

  • Verify no other files reference the old filenamesgrep -r for old names like mouse_example, get_screen_size, report_test.cbl, merge_sort_test, display-test, accept-secure across the repo (READMEs, build scripts, etc.)
  • Decide on read_command_args/ and sub_program/ — These directories contain multiple files where no single file matches the directory name. They were intentionally left unchanged, but you may want them addressed too.
  • Note: report_writer.cbl still has PROGRAM-ID. report-test. — The internal PROGRAM-ID was not changed since it's a runtime identifier, not a file reference. Verify this is acceptable.

Notes

  • All .cbl renames are 100% similarity (no content changes), so no COBOL logic was modified.
  • SQL lint errors in output are pre-existing (SQL files require preprocessing before they can pass syntax checks).

Link to Devin run | Requested by @rdf004


Open with Devin

- Rename redifines/ to redefines/ (fix directory misspelling)
- Rename display_test/display-test.cbl to display_test.cbl (consistent underscores)
- Rename accept/accept-secure.cbl to accept_secure.cbl (consistent underscores)
- Rename merge_sort/merge_sort_test.cbl to merge_sort.cbl (match directory name)
- Rename mouse/mouse_example.cbl to mouse.cbl (match directory name)
- Rename report_writer/report_test.cbl to report_writer.cbl (match directory name)
- Rename screen_size/get_screen_size.cbl to screen_size.cbl (match directory name)
- Update README references to reflect new filenames

Co-Authored-By: Roshan  Fernando <roshandean4@gmail.com>
@devin-ai-integration
Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Copy Markdown
Author

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

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.

0 participants