Update readme badge to use GitHub CI instead of unused Travis #352
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While observing if there are any other mentions for unused Travis I found that it's mentioned here
dataloader/resources/prepublish.sh
Line 7 in 55c33d4
But this script was probably not used at all because for publishing in NPM, it used Github action without
prepublish
scriptdataloader/.github/workflows/release.yml
Lines 8 to 17 in 55c33d4
Should I drop all mentions about prepublish script?
Another observation - another abandoned badge - about code coverage (current badge leads to coveralls, but in workflow codecov is used).
To fix this badge it's needed to copy the link from codecov repo settings https://docs.codecov.com/docs/status-badges
https://app.codecov.io/github/graphql/dataloader/commits?branch=main
Should I drop coveralls badge at all?