-
Notifications
You must be signed in to change notification settings - Fork 892
feat: Rework PAT to use headers only #507
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
Conversation
Co-authored-by: nicoragne <nicoragne@hotmail.fr>
Cursor Agent can help with this pull request. Just @cursor
in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents
@cursoragent c'mon man fix the tests
I refactored the Git utility functions to remove GitHub-specific logic, making them compatible with any Git host. This involved deprecating GitHub-specific checks and exceptions, and switching to
4 tasks completed
Learn more about Cursor Agents |
Co-authored-by: nicoragne <nicoragne@hotmail.fr>
This pull request has merge conflicts that must be resolved before it can be merged.
Generalize PAT handling to support all Git services by removing GitHub-specific logic and passing tokens directly in headers.
Slack Thread