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

Embed test files #2

Merged
merged 2 commits into from
Mar 29, 2022
Merged

Embed test files #2

merged 2 commits into from
Mar 29, 2022

Conversation

carolynvs
Copy link
Member

@carolynvs carolynvs commented Mar 29, 2022

Previously these files were directly in the repository, so we could get away with loading the files as needed. Now that it's used as a library, we have to use go:embed so that we can use the file contents from another repository.

I have also added a GitHub workflow to run the tests.

Previously these files were directly in the repository, so we could get
away with loading the files as needed. Now that it's used as a library,
we have to use go:embed so that we can use the file contents from
another repository.

Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>
@carolynvs carolynvs force-pushed the embed-test-files branch 2 times, most recently from 35fbf2f to df00e44 Compare March 29, 2022 18:05
This adds a github workflow that tests the code when a pull request
is submitted or merged.

Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>
@carolynvs carolynvs merged commit be9946d into main Mar 29, 2022
@carolynvs carolynvs deleted the embed-test-files branch March 29, 2022 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants