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

converted CRLF ending lines to LF ones in api/src/structures/custom_responses.py #199

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

blakkd
Copy link

@blakkd blakkd commented Feb 24, 2025

I had trouble with custom_response.py (which I didn't touch) where windows style ending lines CRLF were making my IDE buggy, so I ended up converting it to Linux style LF.

@blakkd
Copy link
Author

blakkd commented Feb 24, 2025

I'm reallly sorry, I'm quite new to github, and I just saw cherry picking was what I would have to do here, creating 2 branches each for one commit and then make 2 separate PR. Am I right?

I'm really sorry for the mess I'm learning and you pay it!

@fireblade2534
Copy link
Contributor

for one commit and then make 2 separate PR. Am I right?

I'm really sorry for the mess I'm learning and you pay it!

dw its fine lol.

@blakkd
Copy link
Author

blakkd commented Feb 24, 2025

Ok, so should I make 2 separate prs or 1 for both commits (docker + new lines formatting)?

BTW, thanks for accepting being my mentor 🤣 I'll PM you soon to see how we can match our schedules for practicing. I'm so glad, I though it would take months to find someone.

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