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

Follow the user's github theme instead of system #444

Merged
merged 2 commits into from
Nov 30, 2023
Merged

Conversation

pierremtb
Copy link
Collaborator

Fixes #443

@pierremtb pierremtb requested a review from Irev-Dev November 23, 2023 13:13
Copy link
Contributor

@Irev-Dev Irev-Dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!

I can believe I didn't even notice this when I was using the extension 🥴

@pierremtb
Copy link
Collaborator Author

@Irev-Dev Thank you for the review! Let me check why the e2e tests are all timing out in the CI before we merge

@pierremtb
Copy link
Collaborator Author

Oh wow, took me so long to figure this one out. The issue was the GLOBAL_PAT token, that seems to have disappeared from the org actions secrets (but is still present in the dependabots org secrets). I created a GLOBAL_PAT for this repo as a workaround for now

@pierremtb pierremtb merged commit b53062b into main Nov 30, 2023
1 check passed
@pierremtb pierremtb deleted the pierremtb/issue443 branch November 30, 2023 12:53
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.

Follow the user's github theme instead of system
2 participants