Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

chore: fix gosec G115 linting issues for Go 1.24.1 update#17084

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
sreya wants to merge2 commits intomainfromsreya/go-1-24-1-lint-fixes

Conversation

sreya
Copy link
Collaborator

Adds detailed #nosec G115 annotations to resolve gosec linting issues for the Go 1.24.1 upgrade in PR#17035.

This PR adds proper annotations with detailed comments to explain why each integer conversion is safe, which helps the linter understand that these conversions are intentional and the potential overflow risks are handled appropriately.

Part of the work for#17035.

Claudeand others added2 commitsMarch 25, 2025 07:03
Added more detailed #nosec G115 annotations to fix gosec warnings🤖 Generated with [Claude Code](https://claude.ai/code)Co-Authored-By: Claude <noreply@anthropic.com>
- Fix go:build directive spacing in pty_linux.go- Add bounds checks and #nosec annotations for integer conversions- Fix comment alignment and formatting- Address gosec G115 warnings in multiple filesCo-Authored-By: Claude <noreply@anthropic.com>
@github-actionsGitHub Actions
Copy link


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign ourContributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


0 out of2 committers have signed the CLA.
@claude
@user
Claude, User seem not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, pleaseadd the email address used for this commit to your account.
You can retrigger this bot by commentingrecheck in this Pull Request.Posted by theCLA Assistant Lite bot.

@sreyasreya closed thisMar 25, 2025
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsMar 25, 2025
@sreyasreya deleted the sreya/go-1-24-1-lint-fixes branchMarch 25, 2025 07:22
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers
No reviews
Assignees

@sreyasreya

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

1 participant
@sreya

[8]ページ先頭

©2009-2025 Movatter.jp