Skip to content

Conversation

@AntonioGibas
Copy link

removing samples/COBOL/hello_world.cbl, replacing it with example that covers more language usage

Description

Reason for removal:
The repository rules have been updated and explicitly state that generic 'Hello World' examples are not allowed within the samples directory. This commit ensures the COBOL section is compliant with the repository's current guidelines.

Relevant commit:

  • Delete samples/COBOL/hello_world.cbl
  • Add Fibonacci sequence generator and analyzer program

Checklist:

  • I have included a real-world usage sample for all extensions added in this PR:

    • Sample source(s):
      • [URL to each sample source, if applicable]
    • Sample license(s):
  • I have included a change to the heuristics to distinguish my language from others using the same extension.

  • I am fixing a misclassified language

    • I have included a new sample for the misclassified language

Per the contribution guidelines, "Hello World" examples are discouraged as they don't provide enough tokens for the classifier to accurately detect the language.

I am proposing a replacement with a more representative example that demonstrates core COBOL features widely used in legacy systems.
'"Hello world" examples will not be accepted'
Add Fibonacci sequence generator and analyzer program
@AntonioGibas AntonioGibas requested a review from a team as a code owner December 7, 2025 03:35
Copy link
Member

@lildude lildude left a comment

Choose a reason for hiding this comment

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

Add Fibonacci sequence generator and analyzer program

If you're going to remove samples in the name of adhering to the guidelines, you should be adding a real-world sample. Fibonacci examples are just like "Hello world" samples in that no one really uses these outside of learning the language.

The PR template should also point to the source of the real-world sample and state the license of the sample.

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