Skip to content

Document support of .readthedocs.yml #2442

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

Merged
merged 1 commit into from
Oct 13, 2016
Merged

Document support of .readthedocs.yml #2442

merged 1 commit into from
Oct 13, 2016

Conversation

skirpichev
Copy link
Contributor

@skirpichev skirpichev commented Oct 7, 2016

agjohnson added a commit that referenced this pull request Oct 12, 2016
Allauth sent context that wasn't able to serialize with pickle when passing to
Celery. Remove the data, as it wasn't even needed anyways.

Fixes #2442
@agjohnson agjohnson self-assigned this Oct 12, 2016
@agjohnson agjohnson removed their assignment Oct 12, 2016
@agjohnson agjohnson self-assigned this Oct 12, 2016
@agjohnson agjohnson removed their assignment Oct 12, 2016
@skirpichev
Copy link
Contributor Author

@ericholscher, @agjohnson, I think this PR was closed by mistake. Please reconsider and reopen.

@agjohnson
Copy link
Contributor

Indeed, i tagged the wrong issue in my commit. Reopened

@ericholscher
Copy link
Member

Looks good, thanks!

@ericholscher ericholscher merged commit e3815ca into readthedocs:master Oct 13, 2016
@ericholscher
Copy link
Member

We'll get this deployed in the next day or so.

@skirpichev skirpichev deleted the dotfile branch October 13, 2016 18:50
agjohnson pushed a commit that referenced this pull request Oct 18, 2016
Surround .readthedocs.yml with double backticks, to show leading dot.
@skirpichev
Copy link
Contributor Author

@ericholscher, this still doesn't work, like #2368. (Locally it's ok.) I believe, the readthedocs-build package wasn't updated.

Could you do a minor release for readthedocs-build, so we can depend on that version instead of using git repo?

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.

3 participants