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

suggest to seperate tests and examples from lib #47

Closed
retsyo opened this issue Aug 21, 2022 · 1 comment
Closed

suggest to seperate tests and examples from lib #47

retsyo opened this issue Aug 21, 2022 · 1 comment

Comments

@retsyo
Copy link

retsyo commented Aug 21, 2022

when we use nimble install zippy, the message says Downloading https://github.com/guzba/zippy using git. the directory tests takes up 35.1M, so it takes a long time to clone; ok to be honest, I have a bad internet supplier, for most of the time I can't finish git clone. I have to download the zipped source, which will be success sometimes, then install by hand. As a result, it is pain for me to install nim's library.

So, I propose to seperate test and examples from zippy lib, as a result we can get a rapid installation of the lib.

thanks

@guzba
Copy link
Owner

guzba commented Aug 22, 2022

Hello and thanks for the issue. I will not be moving examples or tests out of the repo. It is very important that GitHub Actions can run on PRs and ensure things are working. Keeping this lib working is fundamental and the testing is required for that. As for the examples, they are just some text files and take up no space.

I have updated the tests to use smaller files, reducing the download a bit.

@guzba guzba closed this as completed Aug 22, 2022
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

No branches or pull requests

2 participants