Skip to content
This repository was archived by the owner on Dec 4, 2025. It is now read-only.

Russian translation and setup_path() rework#23

Closed
neijrr wants to merge 22 commits into
tf2classified:mainfrom
neijrr:main
Closed

Russian translation and setup_path() rework#23
neijrr wants to merge 22 commits into
tf2classified:mainfrom
neijrr:main

Conversation

@neijrr
Copy link
Copy Markdown
Contributor

@neijrr neijrr commented Aug 2, 2022

Didn't want it to be together in a single pull request but I made epic branching fail, so here we are.
setup_path() now just loops on custom directory if user says no to installing in sourcemod and current directory, waiting for user to input directory and say yes

@Technochips
Copy link
Copy Markdown
Member

im sorry but this entire commit history looks hellish

Comment thread tf2c_downloader.py Outdated
@Technochips
Copy link
Copy Markdown
Member

Technochips commented Aug 2, 2022

keep one feature per pull request. don't commit unrelated stuff right after posting a pull request. this is just very confusing to review and the commit history is awful to look at.

consider making new forks off the current branch, and redo the features (copy paste?), then make a pull request for each individual forks/features. also don't bother merging 'main' back to your branch unless we tell you to do it.

as far as i can see, this pull request is identical to #19 (same repo) and that one hasn't been able to get pulled because(?) of the confusing commit history, except this one not only tries to fix the same issue that ended up being fixed in a commit in 'main', but tries to do at least three other things.

@Technochips Technochips closed this Aug 2, 2022
@neijrr
Copy link
Copy Markdown
Contributor Author

neijrr commented Aug 2, 2022

So...
Why you closed this pull request?
You and Chloe already told it in Discord before, and #19 was closed because fix in fork main was replaced with code from tf2classic:main (and I didn't like the fix at the time anyway).
There is no new information in your comment and you just close pull request with hours of work.

@newgreenshoot
Copy link
Copy Markdown
Contributor

The code and the work is fine and appreciated. But we would also appreciate if you could split the work into separate pull requests.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants