Skip to content

Hexagonal number sequence #5639

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

Closed
wants to merge 1 commit into from
Closed

Conversation

shri30yans
Copy link
Contributor

@shri30yans shri30yans commented Oct 28, 2021

A hexagonal number sequence is a sequence of figurate numbers where the nth hexagonal number hₙ is the number of distinct dots in a pattern of dots consisting of the outlines of regular hexagons with sides up to n dots, when the hexagons are overlaid so that they share one vertex.

This program returns the hexagonal number sequence of n length.

Describe your change:

  • Add an algorithm?
  • Fix a bug or typo in an existing algorithm?
  • Documentation change?

Checklist:

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized.
  • I know that pull requests will not be merged if they fail the automated tests.
  • This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
  • All new Python files are placed inside an existing directory.
  • All filenames are in all lowercase characters with no spaces or dashes.
  • All functions and variable names follow Python naming conventions.
  • All function parameters and return values are annotated with Python type hints.
  • All functions have doctests that pass the automated testing.
  • All new algorithms have a URL in its comments that points to Wikipedia or other similar explanation.
  • If this pull request resolves one or more open issues then the commit message contains Fixes: #{$ISSUE_NO}.

A hexagonal number sequence is a sequence of figurate numbers where the nth hexagonal number hₙ is the number of distinct dots in a pattern of dots consisting of the outlines of regular hexagons with sides up to n dots, when the hexagons are overlaid so that they share one vertex.

This program returns the hexagonal number sequence of n length.
@shri30yans shri30yans requested a review from Kush1101 as a code owner October 28, 2021 14:54
@ghost ghost added the awaiting reviews This PR is ready to be reviewed label Oct 28, 2021
@ghost
Copy link

ghost commented Oct 28, 2021

Closing this pull request as invalid

@shri30yans, this pull request is being closed as none of the checkboxes have been marked. It is important that you go through the checklist and mark the ones relevant to this pull request. Please read the Contributing guidelines.

If you're facing any problem on how to mark a checkbox, please read the following instructions:

  • Read a point one at a time and think if it is relevant to the pull request or not.
  • If it is, then mark it by putting a x between the square bracket like so: [x]

NOTE: Only [x] is supported so if you have put any other letter or symbol between the brackets, that will be marked as invalid. If that is the case then please open a new pull request with the appropriate changes.

@ghost ghost added the invalid label Oct 28, 2021
@ghost ghost closed this Oct 28, 2021
@ghost ghost removed the awaiting reviews This PR is ready to be reviewed label Oct 28, 2021
@shri30yans
Copy link
Contributor Author

shri30yans commented Nov 6, 2021

Hey @Kush1101,
Could you please consider removing the Invalid Tag.
The bot asks us to read the Contributing guidelines but the guidelines doesn't mention the checklist in any manner.
It is my first time contributing and I wasn't aware of the checklist.
The Invalid Tag disqualifies and bans me from HacktoberFest.

I made a second PR here which was exactly the same but with the [x] (which was accepted).

Please consider removing this tag and adding the checklist to the guidelines to prevent future mishaps.
I would be grateful if you could help me out. Thanks.

This pull request was closed.
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