Skip to content

Extract method - Git._folder_args#1269

Closed
dazza-codes wants to merge 1 commit into
python-poetry:developfrom
dazza-codes:git-folder-args-extract
Closed

Extract method - Git._folder_args#1269
dazza-codes wants to merge 1 commit into
python-poetry:developfrom
dazza-codes:git-folder-args-extract

Conversation

@dazza-codes
Copy link
Copy Markdown
Contributor

@dazza-codes dazza-codes commented Jul 31, 2019

Pure extract-method refactor

checklist

  • doesn't apply here, but adding it for the sake of it
  • Added tests for changed code.
  • Updated documentation for changed code.

@dazza-codes dazza-codes force-pushed the git-folder-args-extract branch from 3938282 to 1f4a482 Compare August 10, 2019 00:10
@brycedrennan brycedrennan added the kind/refactor Pulls that refactor, or clean-up code label Aug 17, 2019
@dazza-codes dazza-codes force-pushed the git-folder-args-extract branch from 1f4a482 to 7aa5eac Compare August 23, 2019 12:25
@dazza-codes
Copy link
Copy Markdown
Contributor Author

PRs are moving very slowly, any plans to merge this?

  • if PRs like this are not merged quickly, there's a risk they fall behind and need extra attention to rebase them again (and again)

@dazza-codes dazza-codes force-pushed the git-folder-args-extract branch from 7aa5eac to eb14d89 Compare September 5, 2019 16:00
@dazza-codes
Copy link
Copy Markdown
Contributor Author

  • rebased on latest develop
  • hope this gets merged soon to keep moving with git-tag feature PRs

@dazza-codes dazza-codes force-pushed the git-folder-args-extract branch from eb14d89 to a2b90bf Compare September 12, 2019 16:41
@dazza-codes
Copy link
Copy Markdown
Contributor Author

rebased on latest develop after master was merged in

  • it's been a month, what does it take to merge this stuff?

@dazza-codes dazza-codes force-pushed the git-folder-args-extract branch from a2b90bf to 3ce58fd Compare September 12, 2019 17:55
@dazza-codes
Copy link
Copy Markdown
Contributor Author

Is anyone going to merge this?

@neersighted
Copy link
Copy Markdown
Member

Hi! Sorry that this has taken so long to review! Unfortunately, there's been enough churn that this is now totally obsolete. I'm closing this, but if you still want to work on/around this aspect of Poetry you are more than welcome to!

@github-actions
Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Feb 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

kind/refactor Pulls that refactor, or clean-up code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants