Skip to content

Use build badges provided by Gitlab #3915

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

Merged
merged 1 commit into from
Jan 8, 2021

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Jan 6, 2021

@UdjinM6 UdjinM6 added this to the 17 milestone Jan 6, 2021
Copy link
Collaborator

@thephez thephez left a comment

Choose a reason for hiding this comment

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

Copy link

@xdustinface xdustinface left a comment

Choose a reason for hiding this comment

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

ACK

Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK

@PastaPastaPasta PastaPastaPasta merged commit 72f2e50 into dashpay:develop Jan 8, 2021
@UdjinM6 UdjinM6 deleted the fixbadges branch July 1, 2021 21:55
PastaPastaPasta added a commit that referenced this pull request May 9, 2025
…r` and `develop`

b8236c8 docs: add build badge from GitHub Actions runs on `master` and `develop` (Kittywhiskers Van Gogh)

Pull request description:

  ## Motivation

  Since [dash#6604](#6604), we haven't seen GitHub Actions runners experience sporadic failures that aren't the result of test failures. In comparison, GitLab CI has been showing greater instability with its runners ([build](https://gitlab.com/dashpay/dash/-/jobs/9960288430), [build](https://gitlab.com/dashpay/dash/-/jobs/9922122668)).

  As GitHub Actions runners are now at coverage parity with GitLab CI, this pull request adds status badges for it in the same vein as [dash#3915](#3915).

  ## How Has This Been Tested?

  ![Build Status](https://github.com/user-attachments/assets/a46647dd-57ba-4da1-a74e-739b80c78c82)

  ## Checklist

  - [x] I have performed a self-review of my own code **(note: N/A)**
  - [x] I have commented my code, particularly in hard-to-understand areas **(note: N/A)**
  - [x] I have added or updated relevant unit/integration/functional/e2e tests **(note: N/A)**
  - [x] I have made corresponding changes to the documentation **(note: N/A)**
  - [x] I have assigned this pull request to a milestone _(for repository code-owners and collaborators only)_

ACKs for top commit:
  UdjinM6:
    LGTM, utACK b8236c8

Tree-SHA512: 705f1be11274689c590029c0695737f2da2e47f2e8591b2a5113de91a2473eef6a24195d5d1918f5dcdaad63d364ac75867ddc3bf2b914d57fba22de1fcad4d8
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.

4 participants