X Tutup
The Wayback Machine - https://web.archive.org/web/20201027222509/https://github.com/TheAlgorithms/C-Plus-Plus/pull/1306
Skip to content

/ C-Plus-Plus

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Create N-bonacci.cpp #1306

Open
wants to merge 15 commits into
base: master
from
Open

Create N-bonacci.cpp #1306

wants to merge 15 commits into from

Conversation

@neha-hasija17
Copy link
Contributor

@neha-hasija17 neha-hasija17 commented Oct 16, 2020 •
edited by gitpod-io bot

This Pull Request is for HacktoberFest 2020

Description of Change

Checklist

  • Added description of change
  • Added file name matches File name guidelines
  • Added tests and example, test must pass
  • Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
  • 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:

@neha-hasija17
Create N-bonacci.cpp
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
a4b795e 
This Pull Request is for HacktoberFest 2020
@Panquesito7
Panquesito7 requested changes Oct 16, 2020
View changes
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Code is not up to the repository standards.
Please read them carefully and follow them.
@neha-hasija17
Update N-bonacci.cpp
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
0466cc5 
This Pull Request is for HacktoberFest 2020
@Panquesito7
Panquesito7 requested changes Oct 16, 2020
View changes
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Documentation is not up to the Doxygen guidelines.
Please read them carefully and follow them.
@neha-hasija17
Update N-bonacci.cpp
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
35f0ffb 
This Pull Request is for HacktoberFest 2020
@neha-hasija17 neha-hasija17 requested a review from Panquesito7 Oct 16, 2020
@Panquesito7
Panquesito7 requested changes Oct 17, 2020
View changes
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Documentation is not up to the Doxygen guidelines.
Please read them carefully and follow them.
math/N-bonacci.cpp Outdated Show resolved Hide resolved
math/N-bonacci.cpp Outdated
*In this code we take N and M as input where M is the number of terms
*to be printed of the N-bonacci series
*/
void N_bonacci(int n,int m){

This comment has been minimized.

Sign in to view
@Panquesito7

Panquesito7 Oct 17, 2020
Member

For how to document functions, please see #962 as a reference. 🙂
math/N-bonacci.cpp Outdated Show resolved Hide resolved
neha-hasija17 and others added 2 commits Oct 17, 2020
@neha-hasija17 @Panquesito7
Update math/N-bonacci.cpp
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
dd63f94 
Co-authored-by: David Leal <halfpacho@gmail.com>
@neha-hasija17
Update N-bonacci.cpp
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
921f61e 
This Pull Request is for HacktoberFest 2020
@neha-hasija17 neha-hasija17 requested a review from Panquesito7 Oct 17, 2020
@neha-hasija17
Copy link
Contributor Author

@neha-hasija17 neha-hasija17 commented Oct 17, 2020

@Panquesito7 please check it now, and let me know the error so I can resolve it asap.
@Panquesito7
Panquesito7 reviewed Oct 18, 2020
View changes
math/N-bonacci.cpp Outdated Show resolved Hide resolved
neha-hasija17 and others added 4 commits Oct 18, 2020
@neha-hasija17 @Panquesito7
Update math/N-bonacci.cpp
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
8a388ec 
Co-authored-by: David Leal <halfpacho@gmail.com>
github-actions github-actions
formatting filenames 8a388ec
2209776
github-actions github-actions
updating DIRECTORY.md
05cc612
github-actions github-actions
clang-format and clang-tidy fixes for 8a388ec
Loading status checks…
e35f026
@neha-hasija17 neha-hasija17 requested a review from Panquesito7 Oct 18, 2020
@Panquesito7
Panquesito7 reviewed Oct 18, 2020
View changes
math/n_bonacci.cpp Outdated Show resolved Hide resolved
math/n_bonacci.cpp Outdated Show resolved Hide resolved
neha-hasija17 and others added 2 commits Oct 18, 2020
@neha-hasija17 @Panquesito7
Update math/n_bonacci.cpp
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
be6636e 
Co-authored-by: David Leal <halfpacho@gmail.com>
github-actions github-actions
clang-format and clang-tidy fixes for be6636e
Loading status checks…
869de3a
@Panquesito7
Panquesito7 requested changes Oct 18, 2020
View changes
math/n_bonacci.cpp Outdated Show resolved Hide resolved
math/n_bonacci.cpp Outdated Show resolved Hide resolved
math/n_bonacci.cpp Outdated Show resolved Hide resolved
math/n_bonacci.cpp
while (t--) {
std::cout << "Enter the values of N and M : ";
std::cin >> n >> m;
N_bonacci(n, m);

This comment has been minimized.

Sign in to view
@Panquesito7

Panquesito7 Oct 18, 2020
Member

Please add self-test cases to ensure the algorithm works as expected. 🙂

This comment has been minimized.

Sign in to view
@neha-hasija17

neha-hasija17 Oct 18, 2020
Author Contributor

please help me with the same

This comment has been minimized.

Sign in to view
@Panquesito7

Panquesito7 Oct 21, 2020
Member

See the typical structure of a program as a reference.
math/n_bonacci.cpp
*In this code we take N and M as input where M is the number of terms
*to be printed of the N-bonacci series
*/
void N_bonacci(int n, int m) {

This comment has been minimized.

Sign in to view
@Panquesito7

Panquesito7 Oct 18, 2020
Member

Missing function documentation.

This comment has been minimized.

Sign in to view
@neha-hasija17

neha-hasija17 Oct 18, 2020
Author Contributor

please help me in resolving this issue

This comment has been minimized.

Sign in to view
@Panquesito7

Panquesito7 Oct 21, 2020
Member

See the typical structure of a program or #962 as a reference.
math/n_bonacci.cpp Show resolved Hide resolved
@neha-hasija17 @Panquesito7
Update math/n_bonacci.cpp
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
547fe78 
Co-authored-by: David Leal <halfpacho@gmail.com>
neha-hasija17 and others added 2 commits Oct 18, 2020
@neha-hasija17 @Panquesito7
Update math/n_bonacci.cpp
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
2b22387 
Co-authored-by: David Leal <halfpacho@gmail.com>
@neha-hasija17 @Panquesito7
Update math/n_bonacci.cpp
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
da9153f 
Co-authored-by: David Leal <halfpacho@gmail.com>
@github-actions github-actions bot force-pushed the neha-hasija17:patch-1 branch 3 times, most recently from 9744e54 to 9acb16e Oct 18, 2020
github-actions github-actions
clang-format and clang-tidy fixes for da9153f
Loading status checks…
7070e79
@github-actions github-actions bot force-pushed the neha-hasija17:patch-1 branch from 9acb16e to 7070e79 Oct 18, 2020
@neha-hasija17 neha-hasija17 requested a review from Panquesito7 Oct 21, 2020
@Panquesito7
Panquesito7 requested changes Oct 21, 2020
View changes
math/n_bonacci.cpp
while (t--) {
std::cout << "Enter the values of N and M : ";
std::cin >> n >> m;
N_bonacci(n, m);

This comment has been minimized.

Sign in to view
@Panquesito7

Panquesito7 Oct 21, 2020
Member

See the typical structure of a program as a reference.
math/n_bonacci.cpp
*In this code we take N and M as input where M is the number of terms
*to be printed of the N-bonacci series
*/
void N_bonacci(int n, int m) {

This comment has been minimized.

Sign in to view
@Panquesito7

Panquesito7 Oct 21, 2020
Member

See the typical structure of a program or #962 as a reference.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Panquesito7 Panquesito7

Assignees
No one assigned
Labels
Proper Documentation Required enhancement good first issue requested changes
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@neha-hasija17 @Panquesito7 @kvedala
You can’t perform that action at this time.
X Tutup