Skip to content

chore: fix C lint errors (issue #6778) #6866

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 1 commit into from
Apr 29, 2025

Conversation

LalitNarayanYadav
Copy link
Contributor

This PR addresses C linting issues detected in example.c as part of the automated lint workflow (run ID: 14583678082).

Changes:

  • Declared strides and shape arrays as const since they are not modified after initialization, resolving the constVariable style warnings.

Files Modified:

  • lib/node_modules/@stdlib/strided/base/unary/examples/c/example.c

Related Issues:

Resolves #6778
Parent: #3235

Linting should now pass for the affected file.

Checklist


@stdlib-js/reviewers

Signed-off-by: Lalit Narayan Yadav <162928571+LalitNarayanYadav@users.noreply.github.com>
@stdlib-bot stdlib-bot added the Needs Review A pull request which needs code review. label Apr 29, 2025
@stdlib-bot
Copy link
Contributor

Coverage Report

Package Statements Branches Functions Lines
strided/base/unary $\color{green}588/588$
$\color{green}+100.00\%$
$\color{green}32/32$
$\color{green}+100.00\%$
$\color{green}6/6$
$\color{green}+100.00\%$
$\color{green}588/588$
$\color{green}+100.00\%$

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

Copy link
Member

@Planeshifter Planeshifter left a comment

Choose a reason for hiding this comment

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

Thanks!

@Planeshifter Planeshifter added the Ready To Merge A pull request which is ready to be merged. label Apr 29, 2025
@stdlib-bot stdlib-bot removed the Needs Review A pull request which needs code review. label Apr 29, 2025
@Planeshifter Planeshifter merged commit 66a381d into stdlib-js:develop Apr 29, 2025
48 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready To Merge A pull request which is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix C lint errors
3 participants