Skip to content
This repository has been archived by the owner on Jul 28, 2022. It is now read-only.

Add tests using themis on github actions #82

Merged
merged 6 commits into from
Feb 4, 2021

Conversation

Matts966
Copy link
Contributor

💪 Summary

fix #80

  • Added tests using themis on github actions.
  • Currently, only the function for getting gh cli token is tested.

🏁 Goals

  • Add test suite for CI
  • Test token registration

@Matts966 Matts966 force-pushed the feature/github-actions branch 4 times, most recently from 8bbc8de to 7c19d86 Compare January 30, 2021 09:04
Copy link
Owner

@skanehira skanehira left a comment

Choose a reason for hiding this comment

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

Thanks!
I have some comment.

.github/workflows/test.yml Outdated Show resolved Hide resolved
.github/workflows/test.yml Outdated Show resolved Hide resolved
.github/workflows/test.yml Outdated Show resolved Hide resolved
autoload/gh/gh.vim Outdated Show resolved Hide resolved
@Matts966 Matts966 force-pushed the feature/github-actions branch from 7c19d86 to 10592f8 Compare January 30, 2021 12:48
@Matts966 Matts966 force-pushed the feature/github-actions branch from 10592f8 to f99cf9f Compare January 30, 2021 12:53
.github/workflows/test.yml Outdated Show resolved Hide resolved
test/gh/gh.vim Outdated Show resolved Hide resolved
@Matts966
Copy link
Contributor Author

Matts966 commented Feb 3, 2021

@skanehira
Added cases without gh and with g:gh_token :)

Copy link
Owner

@skanehira skanehira left a comment

Choose a reason for hiding this comment

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

LGTM
Thanks :)

@skanehira skanehira merged commit 9cfa237 into skanehira:master Feb 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

test: add CI and test
2 participants