-
Notifications
You must be signed in to change notification settings - Fork 36
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
Make our micro-libraries more like other Idiomatic JavaScript micro-libraries #1481
Comments
This has now been done for emitr. |
Although things are now much simpler than they were before, the release process is still too complicated, so I've created #1490 to help with this. |
@andyberry88, you happy for this to be moved to |
👍 |
@dchambers is there a PR that updates the mico-libraries to the latest versions? |
@andyberry88, no, I just went ahead and made the changes in the This is clearly not the right way of doing things, but is probably a consequence of the fact that it's not easy to manage sprints that involve multiple projects. |
@dchambers We need a PR in BRJS to update each library to the latest version. Otherwise we just have a series of microlibraries that have changed but we're not making use of those changes in BRJS. |
Here's a list of improvements to bring us more in line with standard practice these days:
lib
->src
,spec
->test
&target
->dist
.npm test
for running all tests, including a locally run browser test against Firefox, and SauceLab tests against various other browsers.README.md
based documentation only.master
branch instead of agh-pages
branch for simplicity, and get rid of the extra styling, and links to the different presentations of the exact same content.:
as a delimiter to indicate sub-tasks.Additionally, we should provide a
npm run test:browser:debug
task that makes it easy for developers to debug the tests within a browser.The text was updated successfully, but these errors were encountered: