Coder Social home page Coder Social logo

Comments (2)

slurmulon avatar slurmulon commented on May 29, 2024

@gfloyd apoligies for the horrendous response time on this, I don't have a very good reason other than being way too busy and having lots of unread notifications. I will do my best to improve on that front.

Do you think it would make sense to publish the lib dir and remove the preinstall script?

The only reason I didn't do this to begin with is because I don't like checking in compiled library code that parallels my source code changes, it's simply a personal preference I've developed over time. If it makes people's lives' easier to publish lib then I'm more than happy to support it.

It looks like it's because I'm using a newer version of Babel that has an option in my .babelrc that isn't supported by Babel 5.

As of 0.1.13, Babel 6 is now used along with some syntax plugins (es2015, syntax-flow, transform-flow-strip-types). This version also includes a .babelrc file.

Do you think it would make sense to publish the lib dir and remove the preinstall script?

There are two reasons (really one, they relate to the same problem) that I'm using the preinstall script.

  1. This is a remnant from a hacky package.json built around Babel 5. Babel 5 had various packaging issues but the most prominent was that it failed to install before postinstall. After a lot of searching I ended up running npm install --ignore-scripts in preinstall - pretty gross/sluggish but succinct and effective.
  2. I have found, that for a reason I'm not sure of yet, that Babel 6 fails to install completely as well unless you provide npm install --ignore-scripts in preinstall. If you have suggestions on avoiding this problem (preferably without needing to publish lib) then I'm all ears!

Happy to submit a PR if you want!

Please and thank you! Hoping these issues weren't too frustrating for you, and again apologies for the extremely slow response.

from hazy.

slurmulon avatar slurmulon commented on May 29, 2024

As of 0.1.16, hazy no longer requires the host to compile/transclude the library on their own machine. This was mostly due to me doing whatever I could to avoid publishing output folders. After realizing that NPM actually processes your .gitignore file, I addressed the issue by creating an .npmignore that doesn't include lib.

There is no need to build on the host, therefore this exception should no longer be possible. It's definitely no longer an issue after the Babel 6 upgrade in 0.1.13

from hazy.

Related Issues (1)

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.