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.

Sign up for GitHub

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

Jump to bottom

Gulp upgrade #11817

Open
patmmccann opened this issue Jun 18, 2024 · 1 comment · May be fixed by #11782
Open

Gulp upgrade #11817

patmmccann opened this issue Jun 18, 2024 · 1 comment · May be fixed by #11782

Comments

@patmmccann
Copy link
Collaborator

Type of issue

Maintenance

Description

#11782 flags braces as potentially having a memory leak. However the pr breaks e2e tests.

Steps to reproduce

Test page

Expected results

Actual results

Platform details

Other information
@patmmccann patmmccann moved this from Triage to Ready for Dev in Prebid.js Tactical Issues table Jun 18, 2024
@dgirardi
Copy link
Collaborator

I spent some time on this but it's hard to track down. I think it's a problem with webpack-stream not agreeing with gulp 5. Since it's not supposed to change anything about the generated artifacts I think we can postpone it.

@patmmccann patmmccann linked a pull request Jun 27, 2024 that will close this issue
Bump braces and gulp #11782
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Prebid.js Tactical Issues table
Status: Ready for Dev
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bump braces and gulp prebid/Prebid.js
2 participants
@patmmccann @dgirardi