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

Debug command depends on interface rather on implementations #902

Merged
merged 1 commit into from
Nov 22, 2021

Conversation

piotrkreft
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Documented? no
Fixed tickets #901
License MIT

@jdecool
Copy link
Contributor

jdecool commented Nov 18, 2021

Hey @piotrkreft I discover your PR :)

I've just made the same thing in #905 but base on 0.13 because it has introduced an error in one of our projects.
Isn't it interesting to have this fix in the 0.13 branch ?

@piotrkreft
Copy link
Contributor Author

@jdecool you might be right :) I haven't received any feedback from repo maintainers yet though to the issue

@jdecool
Copy link
Contributor

jdecool commented Nov 18, 2021

Let's wait and see what happens

@mcg-web mcg-web merged commit dd40e6a into overblog:master Nov 22, 2021
@mcg-web
Copy link
Member

mcg-web commented Nov 22, 2021

Thank you @piotrkreft !

@piotrkreft piotrkreft deleted the debug-command branch November 22, 2021 15:16
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.

None yet

3 participants