Skip to content

Conversation

jas4711
Copy link
Contributor

@jas4711 jas4711 commented Oct 16, 2025

Hi! I help maintain this package for Debian, and our QA linters noticed some Go source code files had file execute permissions set. I see no reason for this, so this pull requests is a patch to drop them.

Thanks for your consideration,
Simon

@CLAassistant
Copy link

CLAassistant commented Oct 16, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the needs triage Waiting for discussion / prioritization by team label Oct 16, 2025
@hslatman hslatman added this to the v0.28.5 milestone Oct 16, 2025
@hslatman hslatman self-assigned this Oct 16, 2025
@hslatman
Copy link
Member

Hey @jas4711, changes look OK. Can you complete the CLA, if you haven't done so? If that doesn't work for some reason I can recreate the PR, if needed.

@hslatman hslatman self-requested a review October 16, 2025 11:11
Copy link
Contributor

@maraino maraino left a comment

Choose a reason for hiding this comment

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

Thanks @jas4711. That's hard to notice.

@hslatman
Copy link
Member

@jas4711 pinging you again, since you just responded to the other issue. If completing the CLA is not possible for some reason, I can redo the commit/PR.

@hslatman hslatman merged commit 0f591d6 into smallstep:master Oct 22, 2025
15 checks passed
@hslatman
Copy link
Member

Thank you, @jas4711! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs triage Waiting for discussion / prioritization by team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants