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

For #2338 - sphinx.util.compat is deprecated #2339

Closed
wants to merge 1 commit into from
Closed

For #2338 - sphinx.util.compat is deprecated #2339

wants to merge 1 commit into from

Conversation

MaraRinn
Copy link
Contributor

Just removing this dependency doesn't appear to break anything (yet).

Dunbaratu added a commit to Dunbaratu/KOS-1 that referenced this pull request Nov 29, 2018
The repository in the PR is marked as "unknown", meaning
it might have been removed after the PR was made.  So
instead I'm implementing what was in the PR by re-typing it.
@Dunbaratu
Copy link
Member

@Mararin - I'm closing this because I psuedo-merged it by just re-typing the change myself. I had to do this because github isn't telling me the branch the PR came from. It's showing as "unknown repository", which makes me think it may have been deleted after the PR got made.

@Dunbaratu Dunbaratu closed this Nov 29, 2018
@Dunbaratu Dunbaratu added the manual merge-ish The change the PR asks for has happened through some other editing means.. label Nov 29, 2018
@MaraRinn
Copy link
Contributor Author

Yeah, I made some newbie mistakes on the original repository and decided it was going to be easier to just start from scratch, sorry about that.

@Dunbaratu
Copy link
Member

It's all good. I just wanted to make a paper trail documenting that even though this looks like it never got merged, it kinda did in a way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
manual merge-ish The change the PR asks for has happened through some other editing means..
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants