FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Take the pull request comment token from OpenModelica-Hudson by adrpo · Pull Request #341 · OpenModelica/OpenModelicaLibraryTesting · GitHub

Take the pull request comment token from OpenModelica-Hudson - #341

Merged
sjoelund merged 1 commit into
OpenModelica:masterfrom
adrpo:use-openmodelica-hudson-credential
Aug 28, 2026
Merged

Take the pull request comment token from OpenModelica-Hudson#341
sjoelund merged 1 commit into
OpenModelica:masterfrom
adrpo:use-openmodelica-hudson-credential

Conversation

adrpo commented Aug 28, 2026

Copy link
Copy Markdown
Member

The pull request comment stage binds its GitHub token from a github-token
credential:

withCredentials([string(credentialsId: 'github-token', variable: 'GITHUB_TOKEN')]) {
  sh "${report} --comment"
}

The credential that holds a GitHub token on the OpenModelica Jenkins is
OpenModelica-Hudson, so this points withCredentials at that one instead, and
updates the two places that name the credential in prose: the
pull_request_comment parameter description and the README section on
--comment.

Nothing else changes. The token still reaches pr-report.py as GITHUB_TOKEN,
and a run without the credential still writes and publishes the report, with the
summary in the build log.


Generated with Claude Code

The pull request comment stage bound its GitHub token from a `github-token`
credential. The credential that holds a GitHub token on the OpenModelica
Jenkins is `OpenModelica-Hudson`, so point withCredentials at that one and say
so in the parameter description and the README.

Nothing else changes: the token is still exposed to pr-report.py as
GITHUB_TOKEN, and a run without the credential still writes and publishes the
report with the summary in the build log.
sjoelund merged commit a6168fa into OpenModelica:master Aug 28, 2026
7 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL