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

Remove Status Badge from README #3614

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

Harichandra-Prasath
Copy link
Contributor

Description

Currently Parsl has a status badge showing the status of the recent workflow runs. As Parsl has a strict PR check run before merging, if an arbritrary PR fails , the badge ends up with Parsl "failing" which is not true. As of now, there are no events that can target the status of PRs raised against master branch. This PR removes the status badge from the README.rst.

Changed Behaviour

This change doesnt reflect on the functional behaviour of parsl and all programs are expected to work the same.

Fixes

Fixes #3610

Type of change

  • Bug fix

@benclifford benclifford changed the title Removing Status Badge Remove Status Badge Sep 13, 2024
@benclifford benclifford changed the title Remove Status Badge Remove Status Badge from README Sep 13, 2024
@benclifford benclifford merged commit dd9150d into Parsl:master Sep 13, 2024
8 checks passed
@Harichandra-Prasath Harichandra-Prasath deleted the badge-fix branch September 13, 2024 17:03
yadudoc pushed a commit to yadudoc/parsl that referenced this pull request Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants