Skip to content
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

Added documentation for Armstrong Number + Typo fix in Trie.md in /en #251

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

SpiderMath
Copy link
Contributor

@SpiderMath SpiderMath commented Dec 10, 2024

Description

This pull request adds documentation for the algorithm for checking if a number is an Armstrong Number, the link to the entry is here. I plan to add further documentation for other entries but I am starting with low hanging fruit for now.

Checklist

  • A description of the changes in this PR is mentioned above.
  • All the new and existing checks pass.
  • No plagiarized, duplicated, or repetitive documentation that has been directly copied from another source.
  • If it's a new explanation, it contains solid, understandable, and accessible information.
  • I have read the whole contributing guidelines and agree to the Code of Conduct.

Screenshots (if any)

None in this case

Note to reviewers

I'm rather unsure about the quality of my pseudocode, I feel it may be too spammy with comments or code-like to be pseudocode. Please review that bit. And the codespell action fails due to a typo in a completely different file I didn't touch, hence I haven't touched it in this case. I corrected the typo in question, in /en/Data Structures/Tries/trie.md

@SpiderMath
Copy link
Contributor Author

Considering that the codespell is failing due to trie.md, I fixed it (revelant to relevant) and I've edited the title accordingly.

@SpiderMath SpiderMath changed the title Added documentation for Armstrong Number Added documentation for Armstrong Number + Typo fix in Trie.md in /en Dec 10, 2024
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.

1 participant