Add testing/CI through GitHub actions#4
Open
briochemc wants to merge 7 commits into
Open
Conversation
Contributor
Author
|
Bump! Any comments on these? I think this would be a nice step towards registering CMAP.jl! |
Contributor
|
@briochemc it's been a while that I've been busy with some other works. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
This PR adds CI/tests for CMAP.jl through GitHub actions. I'm not entirely sure this is how to do this, but it worked on my fork, so I thought I should make a PR out of it.
For this to work, you will need to add an API key as a "secret" that should be called
CMAP_API_KEYon the repository.During testing, this key is inserted in the
ENVvariable, which the newget_api_key()function checks if the API key is not in the key file (no key file online).I also added some badges like these below to the ReadMe to get a direct indication if something did not pass the tests:
FYI, these badges point to my fork (where I added my API Key as the
CMAP_API_KEYsecret), but this PR's added badges point to simonscmap/CMAP.jl, so they might appear red/failing until you add your ownCMAP_API_KEYsecret on simonscmap/CMAP.jl.