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

Remove withNoNewKeyword enhancer #1372

Merged
merged 1 commit into from
Jan 10, 2019
Merged

Conversation

limonte
Copy link
Member

@limonte limonte commented Jan 9, 2019

Closes #1368

From now on the recommended way to use SweetAlert2 is:

Swal.fire({...options})

@limonte limonte force-pushed the breaking/remove-withNoNewKeyword branch 3 times, most recently from e255bc5 to 3e82c05 Compare January 9, 2019 13:47
@coveralls
Copy link

coveralls commented Jan 9, 2019

Pull Request Test Coverage Report for Build 4486

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 19 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.1%) to 90.72%

Files with Coverage Reduction New Missed Lines %
dist/sweetalert2.js 19 90.72%
Totals Coverage Status
Change from base Build 4484: 0.1%
Covered Lines: 1135
Relevant Lines: 1203

💛 - Coveralls

@limonte
Copy link
Member Author

limonte commented Jan 9, 2019

@zenflow would you have time to take a look at this PR?

Also, do I understand correctly that we don't need withGlobalDefaults as well?

@zenflow
Copy link
Member

zenflow commented Jan 9, 2019

would you have time to take a look at this PR?

Sure I'll have a look later today

Also, do I understand correctly that we don't need withGlobalDefaults as well?

Correct

@limonte limonte force-pushed the breaking/remove-withNoNewKeyword branch from 56f15ef to 502e1fc Compare January 9, 2019 16:17
@limonte
Copy link
Member Author

limonte commented Jan 10, 2019

The bundle is 72B smaller now

In total, we'll make it 1KB smaller than the prev major 🎉

@zenflow I'll merge this PR and continue with other tasks for the next major, feel free to give any feedback on this PR later.

@limonte limonte merged commit 3a8d004 into master Jan 10, 2019
@limonte limonte deleted the breaking/remove-withNoNewKeyword branch January 10, 2019 06:40
limonte added a commit that referenced this pull request Jan 18, 2019
From now on the recommended way to use SweetAlert2 is:
Swal.fire({...options})

BREAKING CHANGE: remove withNoNewKeyword enhancer (#1372)
limonte pushed a commit that referenced this pull request Jan 19, 2019
# [8.0.0](v7.33.1...v8.0.0) (2019-01-19)

* BREAKING CHANGE: remove getButtonsWrapper() ([c93b5e3](c93b5e3))
* BREAKING CHANGE: close() as instance method (#1379) ([2519c17](2519c17)), closes [#1379](#1379)
* BREAKING CHANGE: replace deprecated `jsnext:main` with `module` in package.json (#1378) ([1785905](1785905)), closes [#1378](#1378)
* BREAKING CHANGE: drop Bower support (#1377) ([cb4ef28](cb4ef28)), closes [#1377](#1377)
* BREAKING CHANGE: remove withNoNewKeyword enhancer (#1372) ([f581352](f581352)), closes [#1372](#1372)
* BREAKING CHANGE: remove swal.noop() ([40d6fbb](40d6fbb))
* BREAKING CHANGE: rename $swal2-validationerror -> $swal2-validation-message (#1370) ([9d1b13b](9d1b13b)), closes [#1370](#1370)
* BREAKING CHANGE: inputValidator and preConfirm should always resolve (#1383) ([fc70cf9](fc70cf9)), closes [#1383](#1383)
* BREAKING CHANGE: remove setDefault and resetDefaults (#1365) ([97c1d7c](97c1d7c)), closes [#1365](#1365)
* BREAKING CHANGE: remove extraParams (#1363) ([5125491](5125491)), closes [#1363](#1363)
* BREAKING CHANGE: remove showValidationError and resetValidationError (#1367) ([50a1eff](50a1eff)), closes [#1367](#1367)
* BREAKING CHANGE: remove useRejections and expectRejections (#1362) ([f050caf](f050caf)), closes [#1362](#1362)
* BREAKING CHANGE: dismissReason: overlay -> backdrop (#1360) ([d05bf33](d05bf33)), closes [#1360](#1360)
* BREAKING CHANGE: drop Android 4.4 support (#1359) ([c0eddf3](c0eddf3)), closes [#1359](#1359)

### Features

* **api:** add update() method ([#1186](#1186)) ([348e8b7](348e8b7))

### BREAKING CHANGES

* close() as instance method (#1379)
* drop Android 4.4 support (#1359)
* replace deprecated `jsnext:main` with `module` in package.json (#1378)
* drop Bower support (#1377)
* remove withNoNewKeyword enhancer (#1372)
* remove swal.noop()
* rename $swal2-validationerror -> $swal2-validation-message (#1370)
* inputValidator and preConfirm should always resolve (#1383)
* remove setDefault and resetDefaults (#1365)
* remove extraParams (#1363)
* remove showValidationError and resetValidationError (#1367)
* remove useRejections and expectRejections (#1362)
* dismissReason: overlay -> backdrop (#1360)
* remove getButtonsWrapper()
@limonte
Copy link
Member Author

limonte commented Jan 19, 2019

🎉 This has been shipped in version 8.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

3 participants