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

Rewrite crypto-js in ES6 #236

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Rewrite crypto-js in ES6 #236

wants to merge 5 commits into from

Conversation

Alanscut
Copy link
Contributor

rewrite crypto-js in ES6, and change the test framework to jest.

@Alanscut
Copy link
Contributor Author

Alanscut commented Nov 3, 2019

Hi, Evan @evanvosberg , do you have any time to review the code ?

@SilkageNet
Copy link

希望添加index.d.ts

@entronad
Copy link
Contributor

entronad commented Jan 9, 2020

@SilkageNet
我目前维护一个和 crypto-js 一样的库,有 index.d.ts 支持 TypeScript 应用,用的 ES6+ 支持 module:
crypto-es .

欢迎从 npm 上引入使用。

There is an alternative for this repo, just the same as it and written in ECMAScript 6+ and with TypeScript support: crypto-es .
If you have any ideas, welcome to give issues or PRs !

@evanvosberg
Copy link
Member

@Alanscut I'm so sorry, I still did not find to review this.

@hiradyazdan
Copy link

@evanvosberg this PR can solve a lot of issues with crypto-js size on production builds. It would be great if you find some time to review it.

@Oliver-Fish
Copy link

Any reason why this was abandoned? it would be really great to see this merged

@OptimusPi
Copy link

bump :)

@mw108
Copy link

mw108 commented Oct 19, 2023

Bump

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.

8 participants