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

Update calculation.js: docs update #7650

Closed
wants to merge 1 commit into from

Conversation

gulshan-123
Copy link

Resolves #7155

Changes:

Added clarification that the map function does linear mapping

Screenshots of the change:

PR Checklist

Copy link

welcome bot commented Mar 21, 2025

🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already. And be sure to add yourself to the list of contributors on the readme page!

@gulshan-123
Copy link
Author

@all-contributors please add @gulshan-123 for doc

Copy link
Contributor

@gulshan-123

I've put up a pull request to add @gulshan-123! 🎉

@ksen0
Copy link
Member

ksen0 commented Mar 25, 2025

Hi @gulshan-123 thanks for your effort here, but please note that this is not the recommended usage of the all-contributors bot, and please review the contributor guidelines for future reference, specifically:

You should not file a pull request (or start working on code changes) without a corresponding issue or before an issue has been approved for implementation; that is because the proposed fix may not be accepted, need a different approach entirely, or the actual problem is somewhere else. Any pull requests filed before the issue has been approved for fixing will be closed until approval is given to the issue.

@ksen0 ksen0 closed this Mar 25, 2025
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.

Calculation map() Documentation Enhancement
2 participants