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

chore: replace GH_TOKEN with JF_BOT_TOKEN #1122

Merged
merged 1 commit into from
May 13, 2021
Merged

chore: replace GH_TOKEN with JF_BOT_TOKEN #1122

merged 1 commit into from
May 13, 2021

Conversation

ferferga
Copy link
Member

@ferferga ferferga commented May 12, 2021

GH_TOKEN seems to be used internally by GitHub and it's making some actions we're using in some repos fail (Source)

(PRs made from users of the org are always successful, but PRs that aren't don't work, like Dependabot PRs. This can be checked by looking at the history of server repos and Jellyfin Vue)

The org-wide secret should be renamed to something else to avoid confusion and further replacements or messes in GH's side. This PR replaces GH_TOKEN with JF_BOT_TOKEN, which should be pretty generic.

@jellyfin-bot jellyfin-bot added the ci Pull requests that update Continuos Integration-related code label May 12, 2021
@codecov
Copy link

codecov bot commented May 12, 2021

Codecov Report

Merging #1122 (043c77c) into master (1414ac0) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1122   +/-   ##
=======================================
  Coverage   67.10%   67.10%           
=======================================
  Files          25       25           
  Lines         845      845           
  Branches      143      143           
=======================================
  Hits          567      567           
  Misses        258      258           
  Partials       20       20           

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 1414ac0...043c77c. Read the comment docs.

@heyhippari
Copy link
Contributor

Waiting for the secret change before merging.

cc @anthonylavado

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented May 13, 2021

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 043c77c
Status: ✅  Deploy successful!
Preview URL: https://8bb527cf.jellyfin-vue.pages.dev

View logs

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@ferferga ferferga changed the title chore: replace GH_TOKEN with ORG_PAT chore: replace GH_TOKEN with JF_BOT_TOKEN May 13, 2021
@heyhippari heyhippari merged commit 7e19c91 into master May 13, 2021
@heyhippari heyhippari deleted the pat_token branch May 13, 2021 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Pull requests that update Continuos Integration-related code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants