Skip to content

DOC: Try with new token #16389

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
May 18, 2017
Merged

Conversation

TomAugspurger
Copy link
Contributor

Let's see if this works @jreback.

I think you have to be in the pandas git repo when you travis-encrypt, since that's the Travis build that's accessing the env var.

@jreback
Copy link
Contributor

jreback commented May 18, 2017

just go ahead and merge
as a branch will show the same error (only master can push

@TomAugspurger TomAugspurger merged commit 22ae055 into pandas-dev:master May 18, 2017
@codecov
Copy link

codecov bot commented May 19, 2017

Codecov Report

Merging #16389 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #16389      +/-   ##
==========================================
- Coverage   90.42%    90.4%   -0.03%     
==========================================
  Files         161      161              
  Lines       50997    50997              
==========================================
- Hits        46116    46104      -12     
- Misses       4881     4893      +12
Flag Coverage Δ
#multiple 88.24% <ø> (-0.01%) ⬇️
#single 40.19% <ø> (-0.11%) ⬇️
Impacted Files Coverage Δ
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/common.py 91.05% <0%> (-0.34%) ⬇️
pandas/util/testing.py 80.59% <0%> (-0.2%) ⬇️
pandas/core/frame.py 97.69% <0%> (-0.1%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6dcf2ed...19f022b. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented May 19, 2017

Codecov Report

Merging #16389 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #16389      +/-   ##
==========================================
- Coverage   90.42%    90.4%   -0.03%     
==========================================
  Files         161      161              
  Lines       50997    50997              
==========================================
- Hits        46116    46104      -12     
- Misses       4881     4893      +12
Flag Coverage Δ
#multiple 88.24% <ø> (-0.01%) ⬇️
#single 40.19% <ø> (-0.11%) ⬇️
Impacted Files Coverage Δ
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/common.py 91.05% <0%> (-0.34%) ⬇️
pandas/util/testing.py 80.59% <0%> (-0.2%) ⬇️
pandas/core/frame.py 97.69% <0%> (-0.1%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6dcf2ed...19f022b. Read the comment docs.

@jreback
Copy link
Contributor

jreback commented May 19, 2017

origin	https://@github.com/pandas-docs/pandas-docs-travis.git (fetch)
origin	https://@github.com/pandas-docs/pandas-docs-travis.git (push)
remote: Anonymous access to pandas-docs/pandas-docs-travis.git denied.
fatal: Authentication failed for 'https://@github.com/pandas-docs/pandas-docs-travis.git/'

this was with the previous address (just moved the repo). I think you have to use the token of the remote you are pushing to (e.g. pandas-dev/pandas-docs-travis), and not where you are pushing from

@TomAugspurger
Copy link
Contributor Author

I think I have to update ci/build_docs.sh to use the new address. Let me try that (just pushing to master).

@jorisvandenbossche
Copy link
Member

I moved the docs back for now. But, anyhow, the authentication was still failing ...

pcluo pushed a commit to pcluo/pandas that referenced this pull request May 22, 2017
@TomAugspurger TomAugspurger deleted the new-auth-token branch May 27, 2017 16:36
@TomAugspurger TomAugspurger added this to the 0.21.0 milestone May 27, 2017
stangirala pushed a commit to stangirala/pandas that referenced this pull request Jun 11, 2017
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