Skip to content

DIRECTORY.md not being updated by TravisCI #1460

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

Closed
2 tasks done
chttrjeankr opened this issue Oct 25, 2019 · 12 comments · Fixed by #1461 or #1572
Closed
2 tasks done

DIRECTORY.md not being updated by TravisCI #1460

chttrjeankr opened this issue Oct 25, 2019 · 12 comments · Fixed by #1461 or #1572

Comments

@chttrjeankr
Copy link
Contributor

chttrjeankr commented Oct 25, 2019

@cclauss
Copy link
Member

cclauss commented Oct 25, 2019

If you can make Travis CI write back to the incomming pull request, that would be cool. Writing back to master might put Travis into an infinite loop. @jneeven is doing some magic in lgeiger/black-action#2 but that is based on GitHub Actions, not Travis CI.

@mrvnmchm
Copy link
Member

A fix for the DIRECTORY.md, can't get Travis CI to push back to the initiating PR. #1542

@cclauss
Copy link
Member

cclauss commented Oct 31, 2019

https://github.com/TheAlgorithms/Python/blob/master/.github/workflows/autoblack.yml Is writing changes back to a PR. GitHub Actions Beta ends in two weeks and then it will go live.

@mrvnmchm
Copy link
Member

Nice! That looks very close to the same changes I was making to the travis.yml. When the GH Actions is released I'll make the updates. Thanks @cclauss

@cclauss
Copy link
Member

cclauss commented Oct 31, 2019

See if they will let you into the beta... https://github.com/features/actions

This was referenced Nov 1, 2019
@poyea
Copy link
Member

poyea commented Nov 14, 2019

@cclauss @mrvnmchm GitHub Actions is now available for all public repositories.

@cclauss
Copy link
Member

cclauss commented Nov 14, 2019

#1569

@mrvnmchm
Copy link
Member

Should I make any changes to #1549, or do I just need another peer review?

@cclauss
Copy link
Member

cclauss commented Nov 14, 2019

https://github.com/TheAlgorithms/Python/pull/1570/checks?check_run_id=303560970#step:4:550
remote: Invalid username or password.
fatal: Authentication failed for 'https://github.com/TheAlgorithms/Python/'
##[error]Process completed with exit code 128.

@cclauss
Copy link
Member

cclauss commented Nov 14, 2019

Writing back to a PR is NOT allowed in GitHub Actions. See the other workflow to see how to use push instead.

@chttrjeankr
Copy link
Contributor Author

chttrjeankr commented Nov 14, 2019

This occurs now:
when there's nothing to commit

@cclauss
Copy link
Member

cclauss commented Nov 15, 2019

This should remain open until #1572 lands.

@cclauss cclauss reopened this Nov 15, 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
Development

Successfully merging a pull request may close this issue.

4 participants