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

More explicit setup for tests #5977

Merged
merged 3 commits into from Jul 23, 2019
Merged

Conversation

@stsewd
Copy link
Member

@stsewd stsewd commented Jul 23, 2019

This makes these tests pass in the corporate site, we are relying in the default value (public), but in the corporate site this is private.

stsewd added 3 commits Jul 22, 2019
Default privacy level change in the corporate site,
it fixes a test.
@stsewd
Copy link
Member Author

@stsewd stsewd commented Jul 23, 2019

Wip, I'm still finding more of these :)

@stsewd
Copy link
Member Author

@stsewd stsewd commented Jul 23, 2019

Looks good to go

@stsewd stsewd requested a review from Jul 23, 2019
@stsewd stsewd merged commit 728b790 into readthedocs:master Jul 23, 2019
1 check passed
@stsewd stsewd deleted the more-explicit-setup branch Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants