Skip to content

Add Fibonacci to Fortran#4489

Merged
rzuckerm merged 6 commits intoTheRenegadeCoder:mainfrom
Butenkite:main
Feb 16, 2025
Merged

Add Fibonacci to Fortran#4489
rzuckerm merged 6 commits intoTheRenegadeCoder:mainfrom
Butenkite:main

Conversation

@Butenkite
Copy link
Copy Markdown
Contributor

@Butenkite Butenkite commented Feb 16, 2025

Congrats on taking the first step to contributing to the Sample Programs repository maintained by The Renegade Coder!
For simplicity, please make sure that your pull request includes one and only one contribution.

Please fill one of the sections below as applicable.
Please also add any other relevant information to the Notes section at the bottom.
You may delete or just ignore any other sections.
For more information please refer to our contributing documentation

I Am Adding a New Code Snippet in an Existing Language

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Hey @Butenkite! Thanks for contributing to this project! We are a rather small team, so it may take some time to process this request. In the meantime, there are several ways you can make yourself a part of The Renegade Coder community. For instance, you can:

Thanks for your help!

Copy link
Copy Markdown
Member

@Kaamkiya Kaamkiya left a comment

Choose a reason for hiding this comment

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

Nice work! I just have one change I'd like to request.

Your program checks for ASCII letters, but what about others invalid characters, like "$", "@", etc?

Please check for those characters as well.

Once you've fixed that, I'll merge this PR.

Thank you!

Copy link
Copy Markdown
Collaborator

@rzuckerm rzuckerm left a comment

Choose a reason for hiding this comment

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

This is consistent with how out Fortran samples validate the input.

@rzuckerm rzuckerm merged commit b9b506a into TheRenegadeCoder:main Feb 16, 2025
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.

Add Fibonacci in Fortran

3 participants