Skip to content

style: add missing space #5287

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

Merged
merged 4 commits into from
Feb 19, 2025
Merged

style: add missing space #5287

merged 4 commits into from
Feb 19, 2025

Conversation

rishavtarway
Copy link
Contributor

Resolves #5233.

Description

What is the purpose of this pull request?

This pull request:address the commit comments of (commit ac38037)

Related Issues

Does this pull request have any related issues?

This pull request:

Questions

Any questions for reviewers of this pull request?

Hi, reviewers! Could you please check if I have followed the contributing guidelines properly? If there are any mistakes or areas for improvement in my code or approach, I would appreciate your feedback. I'm open to suggestions for making it better!

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.


@stdlib-js/reviewers

@stdlib-bot stdlib-bot added Statistics Issue or pull request related to statistical functionality. First-time Contributor A pull request from a contributor who has never previously committed to the project repository. Needs Review A pull request which needs code review. labels Feb 18, 2025
@stdlib-bot
Copy link
Contributor

Coverage Report

Package Statements Branches Functions Lines
stats/base/dists/uniform/logpdf $\color{green}291/291$
$\color{green}+100.00\%$
$\color{green}25/25$
$\color{green}+100.00\%$
$\color{green}4/4$
$\color{green}+100.00\%$
$\color{green}291/291$
$\color{green}+100.00\%$

The above coverage report was generated for the changes in this PR.

@stdlib-bot stdlib-bot removed the First-time Contributor A pull request from a contributor who has never previously committed to the project repository. label Feb 19, 2025
@kgryte kgryte changed the title Chore : Address Commit Comments Regarding mathematical and technical Justification chore: address commit comments Feb 19, 2025
@kgryte kgryte changed the title chore: address commit comments style: add missing space Feb 19, 2025
@kgryte kgryte removed the Needs Review A pull request which needs code review. label Feb 19, 2025
@kgryte
Copy link
Member

kgryte commented Feb 19, 2025

As the lint error with the PR title has been resolved, I'll go ahead and merge.

@rishavtarway
Copy link
Contributor Author

@kgryte why the lint_pr_title is not passing the check ?
what should i do for it

@kgryte kgryte merged commit eebd899 into stdlib-js:develop Feb 19, 2025
59 of 60 checks passed
@rishavtarway rishavtarway deleted the fix/#5233 branch February 19, 2025 04:46
@rishavtarway
Copy link
Contributor Author

Thank you sir

@kgryte
Copy link
Member

kgryte commented Feb 19, 2025

@rishavtarway
Copy link
Contributor Author

Sure sir i will adhere with best practices from next contribution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Statistics Issue or pull request related to statistical functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Address commit comments (commit ac38037)
3 participants