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

Refactor tr_torrentFindFile2 #921

Merged
merged 9 commits into from Apr 27, 2020

Conversation

RobCrowston
Copy link
Contributor

No description provided.

Copy link
Member

@ckerr ckerr left a comment

Choose a reason for hiding this comment

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

Looks like a reasonable change. Thanks for the refactor!

@RobCrowston
Copy link
Contributor Author

Is there anything else I need to do to get these PRs in?

@RobFog
Copy link
Contributor

RobFog commented Apr 27, 2020

Rebase to remove all the merge commits?

@ckerr
Copy link
Member

ckerr commented Apr 27, 2020

Rebase to remove all the merge commits?

I didn't do anything special -- GitHub has a button to get the PR in sync with master so that it can be merged, and I pushed that button. 😐

When committing to master, I'll use squash-and-merge to remove some of that noise in the commit log.

@ckerr ckerr merged commit 6bb8b2e into transmission:master Apr 27, 2020
@ckerr ckerr added type:refactor A code change that neither fixes a bug nor adds a feature scope:core labels Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope:core type:refactor A code change that neither fixes a bug nor adds a feature
Development

Successfully merging this pull request may close these issues.

None yet

4 participants