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

Add coveralls badge back to README #2978

Merged
merged 2 commits into from
Jan 28, 2021

Conversation

jcalcaben
Copy link
Contributor

Description

Adds the Coveralls badge back to the README

Related Issue

Closes PWA-1012

Acceptance

Anyone on the team

Verification Stakeholders

Anyone on the team

Specification

Verification Steps

  1. View the README.md file
  2. Verify Coveralls badge appears at the top with accurate coverage

Screenshots / Screen Captures (if appropriate)

Checklist

  • I have added tests to cover my changes, if necessary.
  • I have added translations for new strings, if necessary.
  • I have updated the documentation accordingly, if necessary.

@jcalcaben jcalcaben added the version: Patch This changeset includes backwards compatible bug fixes. label Jan 27, 2021
@PWAStudioBot
Copy link
Contributor

PWAStudioBot commented Jan 27, 2021

Messages
📖

Associated JIRA tickets: PWA-1012.

📖 DangerCI Failures related to missing labels/description/linked issues/etc will persist until the next push or next nightly build run (assuming they are fixed).
📖

Access a deployed version of this PR here. Make sure to wait for the "pwa-pull-request-deploy" job to complete.

Generated by 🚫 dangerJS against 35ba0b7

@dpatil-magento
Copy link
Contributor

Confirmed with CI build https://coveralls.io/builds/36650787

@dpatil-magento dpatil-magento merged commit c6fab8a into develop Jan 28, 2021
@dpatil-magento dpatil-magento deleted the jimothy/pwa-1012_add-coveralls-badge branch January 28, 2021 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Progress: done version: Patch This changeset includes backwards compatible bug fixes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants