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

Report QA site build status to GitHub #3576

Merged
merged 14 commits into from
Jul 22, 2021
Merged

Report QA site build status to GitHub #3576

merged 14 commits into from
Jul 22, 2021

Conversation

dafeder
Copy link
Member

@dafeder dafeder commented Jul 22, 2021

Report a second status to github from Jenkins with a direct link to the QA site. Also removes the "Check QA site" stage which seems to be unnecessary.

Shoutout to @woodt for the new function added to the Jenkinsfile.

QA Steps

Click "details" link in the new continuous-integration/jenkins/qa-site status in this PR and confirm it links directly to a site.

Jenkinsfile Show resolved Hide resolved
@dafeder dafeder marked this pull request as ready for review July 22, 2021 17:57
@dafeder dafeder requested a review from grugnog July 22, 2021 18:00
Copy link
Member

@grugnog grugnog left a comment

Choose a reason for hiding this comment

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

Everything else looks good and I can see the status

Jenkinsfile Show resolved Hide resolved
@dafeder dafeder merged commit 0a6527c into 2.x Jul 22, 2021
@dafeder dafeder deleted the jenkins-status branch July 22, 2021 20:07
clayliddell pushed a commit that referenced this pull request Jul 26, 2021
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