-
Notifications
You must be signed in to change notification settings - Fork 9
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
Cool project - add some unit tests! #3
Comments
+1 |
Totally agreed. I plan to to hook it up to travis soon. My biggest priority right now is to successfully build and run Sentry (https://getsentry.com/welcome/): a big, very zip unsafe app. I'm almost there and once I'm done, I plan to use Sentry build as a canary to detect regressions. I hope to add some regular unit tests too. |
Thanks, Marcin! I'm excited to follow your progress. |
I'm interested in playing around with exxo to create stand-alone easy-to-distribute releases of EasyBuild (http://hpcugent.github.io/easybuild). I need to fix some known issues w.r.t. making it zip-safe however, cfr. easybuilders/easybuild-framework#1399 |
@boegel I added few zip safety hacks in I curious if new hack already help with EasyBuild framework. Of course, proper zip safety fix is always better. |
Btw. you can read about the hacks here. |
@mbachry: thanks for the update, this looks very useful indeed! I'm interested in making EasyBuild truly zip-safe, but this can certainly help to play around with exxo earlier. @fgeorgatos: maybe this is something for you to play around with next week during the hackathon/user meeting? |
This piqued my interest, and I was about to start trying it out, but then I noticed you haven't written any unit tests yet! I know you just started this recently, but if you're at all interested in getting support/code contributions, and adoption, you must add some test coverage!
I'm excited to see where you take this project. Seems like a good approach as an alternative to PEX.
The text was updated successfully, but these errors were encountered: