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

chai-spies.js needs a rebuild on master #76

Closed
shellscape opened this issue Aug 16, 2017 · 9 comments
Closed

chai-spies.js needs a rebuild on master #76

shellscape opened this issue Aug 16, 2017 · 9 comments

Comments

@shellscape
Copy link

Come on folks, this one is a baddie. Because of the slow release cycle of this module, folks are depending on installing from master (which is bad enough in of itself). Please get this file up to date with the rest of the repo on master.

@shellscape
Copy link
Author

And because it's been a few years since that file was updated, folio has since broken. The fix so that builds again is here: logicalparadox/folio#4

@keithamus
Copy link
Member

Hey @shellscape thanks for the issue.

If you'd like, please feel free to run a build of this package and submit a PR, I'll be happy to merge and publish 😄

@shellscape
Copy link
Author

@keithamus I'm already tracking master changes on this fork and publishing to chai-spies-next because of the slow release cycle of the module. Please feel free to cherry pick from that fork to get up to speed here in upstream.

I'm sympathetic to the amount of time open source projects take to maintain (especially since I joined with the webpack team for some of their repos - it's bananas), but with a very slow release cycle, oodles of people depending on master, long since dated files, and a broken build process, perhaps a call for maintainers would be in order. It could really use some new ❤️ .

@keithamus
Copy link
Member

@shellscape we are always happy to add new maintainers - typically after one or two successful PRs a contributor makes, we will bring them on as a maintainer to help with the project.

If you would like, as you're maintaining the fork - I would be happy to add you as a maintainer to help build & push features through. I have very little time for coding or releasing but I'm always around to review PRs and work on the issue tracker.

@shellscape
Copy link
Author

@keithamus I'd be happy to help the project out. But what's the publish path? Without a nimble way to publish new versions to NPM I'm not sure that it's time well spent (and I mean absolutely no disrespect by that).

@stalniy
Copy link
Contributor

stalniy commented Aug 16, 2017

@keithamus what do you think if I replace folio with something more modern (e.g. rollup or browserify) and replace Makefile with npm scripts?

Then I will rebuild and send a PR

@shellscape
Copy link
Author

rollup + npm script is a win.

@stalniy
Copy link
Contributor

stalniy commented Sep 12, 2017

@keithamus what are the next steps for release to happen? Could you please define a milestone on github? So, I could understand and help to move things forward :)

Thanks!

@keithamus
Copy link
Member

@stalniy if we're happy to release with the current state as-is, then we could make a PR to compile chai-spies.js, change the package.json version - then when merged we can tag the commit and make a release.

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

3 participants