fix(rollup-plugin): replace package rollup-plugin-terser by @rollup/plugin-terser

Brought to you by: tomkozak

#3170 fix(rollup-plugin): replace package rollup-plugin-terser by @rollup/plugin-terser

Status: closed

Owner: nobody

Labels: None

Updated: 2024-04-16

Created: 2023-01-30

Creator: Anonymous

Private: No

Originally created by: ansidev

Prior to creating a pull request, please follow all the steps in the contributing guide.

Fixes [#3136], [#3144]

Description of what's changed/fixed.

Replaces the package rollup-plugin-terser by @rollup/plugin-terser since rollup-plugin-terser is deprecated.

Related

Tickets: ~~#3136~~
Tickets: ~~#3143~~
Tickets: ~~#3144~~
Tickets: ~~#3200~~
Tickets: ~~#3228~~
Tickets: ~~#3301~~

Discussion

Anonymous - 2023-06-27

Originally posted by: Artur-

Could this be merged now that there is a workbox 7, @tropicadri ?

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2023-06-28

Originally posted by: ansidev

Rebased onto v7!

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2023-08-08

Originally posted by: alangdm

Any chance we could see this get merged?
I'm getting errors using tools that depend on this package that have already removed the deprecated terser

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2023-08-08

Originally posted by: ansidev

It seems that maintainers are laid-off 🙃

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2023-08-08

Originally posted by: thepassle

This is also causing issues for consumers of rollup-plugin-workbox apparently, so indeed would be great to get this merged :)

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2023-08-31

Originally posted by: DamianGlowala

@philipwalton, please? 🙏

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2023-09-28

Originally posted by: davidtranjs

Please help to check this PR, it would help a lot 🥺

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2023-10-19

Originally posted by: pwbriggs

Fixes [#3136], [#3144]

Also fixes [#3142], [#3154], [#3200], and [#3228]. @googlechrome @jeffposnick @philipwalton can someone please merge this PR?

Related

Tickets: ~~#3136~~
Tickets: ~~#3142~~
Tickets: ~~#3144~~
Tickets: ~~#3154~~
Tickets: ~~#3200~~
Tickets: ~~#3228~~

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2024-03-18

Originally posted by: Ten-Taken

Would love to see a fix as well. I think this is now causing issues with rollup's dynamic imports. I suspect an unpinned dependency in the older rollup repos.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2024-03-20

Originally posted by: ansidev

I guess Google does not care about this project anymore, the latest commit is pushed 10 months ago.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2024-04-16

Ticket changed by: tropicadri

status: open --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.