[PR #1232] [MERGED] Update fibonacci.c #1825

Open
opened 2026-01-29 15:25:03 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/1232
Author: @niranjank2022
Created: 3/9/2023
Status: Merged
Merged: 4/20/2023
Merged by: @Panquesito7

Base: masterHead: master


📝 Commits (10+)

  • 19cf238 Improve the documentation of factorial.c
  • 6496432 Improve the documentation of fibonacci.c
  • 6c98363 Update starting terms of fibonacci as 0 and 1
  • 6465e90 Update math/fibonacci.c
  • 5738231 docs: Documenting the code
  • 2c0da3e test: Add test
  • 3ab82ea Merging the versions
  • 4139661 fix: fix the test expression
  • 3c0fcaa feat: Restrict non-integer inputs
  • 0864082 fix: Change atoi() to sscanf()

📊 Changes

1 file changed (+113 additions, -12 deletions)

View changed files

📝 math/fibonacci.c (+113 -12)

📄 Description

Description of Change

Included some documentation and modified the base condition and return data-type of fib() function.

Checklist

  • Added description of change
  • Relevant documentation/comments is changed or added
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:
I am a beginner. If I need to change or improve anything, kindly say. I would rectify it.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/TheAlgorithms/C/pull/1232 **Author:** [@niranjank2022](https://github.com/niranjank2022) **Created:** 3/9/2023 **Status:** ✅ Merged **Merged:** 4/20/2023 **Merged by:** [@Panquesito7](https://github.com/Panquesito7) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (10+) - [`19cf238`](https://github.com/TheAlgorithms/C/commit/19cf23865a2690f351e44f6eee43b9cb4f57b54d) Improve the documentation of factorial.c - [`6496432`](https://github.com/TheAlgorithms/C/commit/64964320c454873c405bea54f959ca6f5457589d) Improve the documentation of fibonacci.c - [`6c98363`](https://github.com/TheAlgorithms/C/commit/6c98363e1b20aaa596f19248768e0b46db1c4549) Update starting terms of fibonacci as 0 and 1 - [`6465e90`](https://github.com/TheAlgorithms/C/commit/6465e90a5194bcd5cdbbe58a7ce88145599a869c) Update math/fibonacci.c - [`5738231`](https://github.com/TheAlgorithms/C/commit/5738231fe8fccb2277ce9bfc971a48fca03c6b44) docs: Documenting the code - [`2c0da3e`](https://github.com/TheAlgorithms/C/commit/2c0da3e03846f99c62d54dea0bb62c3251634505) test: Add test - [`3ab82ea`](https://github.com/TheAlgorithms/C/commit/3ab82ea22ca339021047f761512914e680160e83) Merging the versions - [`4139661`](https://github.com/TheAlgorithms/C/commit/4139661b3213b8cb8c4a9c0914c4b7f003718c30) fix: fix the test expression - [`3c0fcaa`](https://github.com/TheAlgorithms/C/commit/3c0fcaa3984dbc248ca4587c8885664e6f8c65cb) feat: Restrict non-integer inputs - [`0864082`](https://github.com/TheAlgorithms/C/commit/08640825a752f6d241f0bdbe30aa5b62e9822bc6) fix: Change atoi() to sscanf() ### 📊 Changes **1 file changed** (+113 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `math/fibonacci.c` (+113 -12) </details> ### 📄 Description #### Description of Change Included some documentation and modified the base condition and return data-type of fib() function. #### Checklist <!-- Remove items that do not apply. For completed items, change [ ] to [x]. --> - [x] Added description of change - [x] Relevant documentation/comments is changed or added - [x] PR title follows semantic [commit guidelines](https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md#Commit-Guidelines) - [x] Search previous suggestions before making a new one, as yours may be a duplicate. - [x] I acknowledge that all my contributions will be made under the project's license. Notes: <!-- Please add a one-line description for developers or pull request viewers --> I am a beginner. If I need to change or improve anything, kindly say. I would rectify it. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 15:25:03 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#1825