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

chore(cleanup): update and make the tests pass #29

Open
wants to merge 27 commits into
base: master
Choose a base branch
from

Conversation

toutpt
Copy link

@toutpt toutpt commented Aug 13, 2020

Hi !

So FYI I have decided after more than a month without any news to fork it.
You will find a first release which correspond to this PR https://github.com/toutpt/module-to-cdn/releases/tag/v3.2.0

And here is the package: https://www.npmjs.com/package/@talend/module-to-cdn

This release is the first one as a fork of module-to-cdn.

Main idea is to leverage a CDN in current modern build tools.
This module is the first step.

Features added:

  • remove some useless dot files
  • add support for a var (global) that is changed (this is the case for history and rxjs
  • merge back all PRs while fixing the tests (ag-grid, d3, bootstrap, aframe)
  • configure resolution url to plug other CDN (private / companies)
  • remove unpkg from the modules.json file

Fix:

  • update all dependencies and dev dependencies
  • add package-lock.json This is import to track security issues and ensure an install can be reproduced.
  • Make all tests passed (updating all existing libs)

Chore:

  • Tests: ensure next can be unsupported without braking the tests
  • keep only nodejs lts versions, 10 && 12 (last version of ava do not work with node prior to 10)
  • add LIMIT env var for the tests so you can focus on one module
  • add axios-retry so if a request fail during the test it will retry 3 times
  • Improve CI by adding cache so tests are runned in less than a minutes and are far more consistents

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

Successfully merging this pull request may close these issues.

2 participants