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

Do not emit type arguments twice #7569

Merged
merged 1 commit into from Apr 27, 2020

Conversation

eric-wieser
Copy link
Contributor

Fixes gh-7567

Feature or Bugfix

  • Bugfix

@eric-wieser
Copy link
Contributor Author

Ping @tk0miya, CI passes now - sorry for not letting you know I was working on the same thing as you before you spent the time!

@tk0miya
Copy link
Member

tk0miya commented Apr 27, 2020

Don't mind. I feel your implementation is much better than mine :-)

@tk0miya tk0miya merged commit a527c2a into sphinx-doc:3.0.x Apr 27, 2020
tk0miya added a commit that referenced this pull request Apr 27, 2020
@tk0miya tk0miya modified the milestones: 3.1.0, 3.0.4 Apr 27, 2020
@tk0miya
Copy link
Member

tk0miya commented Apr 27, 2020

oh, I did not noticed the target branch is 3.0.x.

tk0miya added a commit that referenced this pull request Apr 27, 2020
@eric-wieser
Copy link
Contributor Author

Whoops, sorry about that.

tk0miya added a commit that referenced this pull request Apr 27, 2020
@tk0miya
Copy link
Member

tk0miya commented Apr 27, 2020

don't mind! I think this is not a critical, but stable.

@eric-wieser
Copy link
Contributor Author

I assume this will end up in 3.1.0 as well?

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants