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

Move test program to submodule #86

Merged
merged 7 commits into from
May 2, 2023
Merged

Move test program to submodule #86

merged 7 commits into from
May 2, 2023

Conversation

slhmy
Copy link
Member

@slhmy slhmy commented Apr 25, 2023

Solve #85.

Additionally fixed SETUP_ENV bash.
Dev env should be usable for cargo test if you run git submodule update --init --recursive.

@slhmy slhmy changed the title Slhmy/move test Move test program to submodule Apr 25, 2023
@akamya997
Copy link
Contributor

It seems good, but I think some guides should be added to README.md so that it could help others build the developing environment from scratch.
BTW, though it is enough now, but I hope that we can remove the binary files as well as the testlib.h in the future.

.github/workflows/rust_build.yml Outdated Show resolved Hide resolved
.github/workflows/rust_build.yml Outdated Show resolved Hide resolved
@slhmy
Copy link
Member Author

slhmy commented May 2, 2023

I'm merging this PR right now, for the known issues can be fixed later.

@slhmy slhmy merged commit a3fc83b into main May 2, 2023
@slhmy slhmy deleted the Slhmy/move-test branch June 7, 2023 07:36
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.

3 participants