-
-
Notifications
You must be signed in to change notification settings - Fork 500
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
Update package, config, issue template and format #743
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
55e0f84
Explain me why need this ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you know about preprocess ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've heard about it, but I don't know the details lol. But if we remove this, it would be good because we wouldn't need to use quotation marks. Am I mistaken?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have to stop doing this kind of thing, if we put that it is for a reason, I asked you to test before making PR!
I think tslib is used in production |
It’s a 'devDependencies' otherwise you would see it called in the codes... |
rollbacked |
Hi, I'm a bit confused about the purpose of this PR 🤔. Could you provide more details or clarify what changes are being made and why? |
Sorry I can't accept your PR Only updating package |
No description provided.