Skip to content

algorithm: Pronic number - #595

Merged
raklaptudirm merged 19 commits into
TheAlgorithms:masterfrom
itsAkshayDubey:algorithm-pronic-number
Nov 11, 2022
Merged

raklaptudirm merged 19 commits into
TheAlgorithms:masterfrom
itsAkshayDubey:algorithm-pronic-number

Conversation

@itsAkshayDubey

Copy link
Copy Markdown
Contributor

This PR contains commits related to pronic number implementation.

Thanks.

Comment thread math/pronicnumber.go Outdated
Comment thread math/pronicnumber.go Outdated
raklaptudirm
raklaptudirm previously approved these changes Oct 28, 2022
Comment thread math/pronicnumber_test.go
Comment thread math/pronicnumber.go Outdated
Comment thread math/pronicnumber.go Outdated
Comment thread math/pronicnumber.go Outdated
Comment thread math/pronicnumber.go Outdated
@itsAkshayDubey
itsAkshayDubey marked this pull request as draft November 4, 2022 03:56
@itsAkshayDubey
itsAkshayDubey marked this pull request as ready for review November 7, 2022 18:32
@itsAkshayDubey

Copy link
Copy Markdown
Contributor Author

@tjgurwara99 and @vil02 I added all the suggested changes.
Kindly have a look.

Thanks,
Akshay

@vil02 vil02 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is some overflow issue. Interesting.

Comment thread math/pronicnumber_test.go Outdated
@itsAkshayDubey
itsAkshayDubey requested review from raklaptudirm, tjgurwara99 and vil02 and removed request for tjgurwara99 and vil02 November 8, 2022 06:08

@vil02 vil02 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me, besides the name in the test cases.

Comment thread math/pronicnumber_test.go
Comment thread math/pronicnumber_test.go Outdated
Comment thread math/pronicnumber.go Outdated
@itsAkshayDubey
itsAkshayDubey requested review from raklaptudirm, tjgurwara99 and vil02 and removed request for raklaptudirm, tjgurwara99 and vil02 November 9, 2022 06:02
Comment thread math/pronicnumber.go
@raklaptudirm
raklaptudirm merged commit f2296d5 into TheAlgorithms:master Nov 11, 2022
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.

4 participants