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

fix: remove unused indenting of stdout/stderr #130

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

rhizoome
Copy link
Collaborator

I recently copy-pasted from pyaptly and found an issue.

@rhizoome rhizoome force-pushed the jlf-indent branch 2 times, most recently from 0a7b98a to fb55e03 Compare February 26, 2025 15:31
@rhizoome
Copy link
Collaborator Author

rhizoome commented Feb 26, 2025

@Melkor333 oh my. now you have to approve every CI run. that is going to be annoying.

@Melkor333
Copy link
Contributor

Gave you an invite for write access. 😇

@rhizoome
Copy link
Collaborator Author

rhizoome commented Feb 26, 2025

Gave you an invite for write access. 😇

Thanks. Makes sense since notification-wise github still thinks I am an owner of this project. I get everything and can't change it.

Copy link
Contributor

@Melkor333 Melkor333 left a comment

Choose a reason for hiding this comment

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

hm weird, i thought github notification settings allow you basically everything, even to disable notifications for your own repos completely.. But I might be wrong.

Anyway, thanks a lot for the two fixes! And nice that you could benefit from it being OSS 😇

@Melkor333 Melkor333 merged commit e783ae9 into adfinis:main Mar 3, 2025
2 checks passed
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.

2 participants