Skip to content

feat: using token authorization for styles on Github to be able to us…#603

Merged
andreoliwa merged 2 commits into
andreoliwa:developfrom
danygielow:fix/github-auth
Aug 19, 2023
Merged

feat: using token authorization for styles on Github to be able to us…#603
andreoliwa merged 2 commits into
andreoliwa:developfrom
danygielow:fix/github-auth

Conversation

@danygielow
Copy link
Copy Markdown
Contributor

@danygielow danygielow commented Jun 9, 2023

…e all types of Github tokens

Issues fixed by this pull request:
Fix #602

Proposed changes

  1. Using Token Authorization instead of Basic Authorization when downloading styles from Github

Checklist

  • Read the contribution guidelines
  • Run make locally before pushing commits
  • Add tests for the relevant parts:
    • API
    • CLI
    • flake8 plugin (normal mode)
    • flake8 plugin (offline mode)
  • Write documentation when there's a new API or functionality

@andreoliwa
Copy link
Copy Markdown
Owner

andreoliwa commented Jun 9, 2023

Thank you for the PR!
I'll try this as soon as I can and get back to you.

@andreoliwa
Copy link
Copy Markdown
Owner

andreoliwa commented Aug 19, 2023

@danygielow I have tested your branch with personal access tokens (both classic and beta), and they still work.

I don't have a GitHub App setup to test manually but I trust it works for you.

I'm merging this.
Thanks again. 🚀

@andreoliwa andreoliwa merged commit 5d23b6c into andreoliwa:develop Aug 19, 2023
github-actions Bot pushed a commit that referenced this pull request Aug 19, 2023
@semantic-release-bot
chore(release): 0.34.0 [skip ci]
309379e 
# [0.34.0](v0.33.2...v0.34.0) (2023-08-19)

### Features

* accept all types of GitHub tokens (personal and apps) ([#603](#603)) ([5d23b6c](5d23b6c))
* drop support for Python 3.7 ([65178f3](65178f3))
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 19, 2023

🎉 This PR is included in version 0.34.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions Bot added the released Feature/fix is released label Aug 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

released Feature/fix is released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Allow Github Apps Access Tokens in Github URLs

2 participants

@danygielow @andreoliwa