Skip to content
This repository was archived by the owner on Feb 22, 2024. It is now read-only.

Move test suite to its own repository #23

Merged
merged 1 commit into from
Jan 5, 2021
Merged

Move test suite to its own repository #23

merged 1 commit into from
Jan 5, 2021

Conversation

glyn
Copy link
Contributor

@glyn glyn commented Jan 5, 2021

Fixes #3

@glyn glyn self-assigned this Jan 5, 2021
@glyn glyn requested a review from gregsdennis January 5, 2021 08:49
@gregsdennis
Copy link
Collaborator

gregsdennis commented Jan 5, 2021

Where's the new repo? Probably want to set that up first (if it's not already) and update the build to point there for the test suite.

@glyn
Copy link
Contributor Author

glyn commented Jan 5, 2021

Where's the new repo? Probably want to set that up first (if it's not already) and update the build to point there for the test suite.

It's here where the submodule points to. ;-) The test failure above was due to the github action grabbing the code without its submodule. I need to update the action config.

Copy link
Collaborator

@gregsdennis gregsdennis left a comment

Choose a reason for hiding this comment

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

LGTM. Get the build running properly and merge away!

@glyn glyn merged commit a8e511c into jsonpath-standard:main Jan 5, 2021
@glyn glyn deleted the move-cts-to-new-repo branch January 5, 2021 10:01
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 suite should be split into its own repo
2 participants