Coder Social home page Coder Social logo

Comments (9)

nickytonline avatar nickytonline commented on May 5, 2024 1

I created PR #86 to fix this. Feedback welcome.

from react-slingshot.

coryhouse avatar coryhouse commented on May 5, 2024

You just need to add at least one .spec file under /src. Alternatively, you can disable Mocha. This error occurs when Mocha can't find any tests.

Thus, I'm torn on whether to consider this a bug or a feature. :)

from react-slingshot.

freeman29 avatar freeman29 commented on May 5, 2024

Thanks Cory.. The slingshot is awesome. I will always use this on my projects instead of gulp.

from react-slingshot.

coryhouse avatar coryhouse commented on May 5, 2024

I'd like to provide a friendly message when no tests are found, but I haven't found a way to suppress this message. Open to ideas.

from react-slingshot.

coryhouse avatar coryhouse commented on May 5, 2024

I'm considering adding a check in the build that runs before Mocha. It would look for test files and if none exist, it would stop the build and message to the user to either:
a. Add at least one test
b. Disable running tests

This way, this error doesn't confuse others. Sound good?

from react-slingshot.

freeman29 avatar freeman29 commented on May 5, 2024

That would be great. are you using terminal commands on the checking or
webpack?

On 20 February 2016 at 04:37, Cory House [email protected] wrote:

I'm considering adding a check in the build that runs before Mocha. It
would look for test files and if none exist, it would stop the build and
message to the user to either:
a. Add at least one test
b. Disable running tests

This way, this error doesn't confuse others. Sound good?


Reply to this email directly or view it on GitHub
#40 (comment)
.

from react-slingshot.

coryhouse avatar coryhouse commented on May 5, 2024

Decided to leave this as is. See PR #86 for conversation on why. That said, thanks again to @nickytonline for continued support! 👍

from react-slingshot.

nickytonline avatar nickytonline commented on May 5, 2024

@freeman29, the PR #86 won't be merged, but the functionality you want is there, so feel free to grab it.

from react-slingshot.

freeman29 avatar freeman29 commented on May 5, 2024

I'll check it this weekend then I'll give my feedback. Thanks @nickytonline .

from react-slingshot.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.