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

Unpin webpack version #275

Merged
merged 1 commit into from
Jan 14, 2022
Merged

Unpin webpack version #275

merged 1 commit into from
Jan 14, 2022

Conversation

artonge
Copy link
Contributor

@artonge artonge commented Jan 12, 2022

webpack/webpack#14628 is now merged, it should be safe to unpin the webpack version.

Let's wait until the equivalent server's PR is all green: nextcloud/server#30632

Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge added the dependencies Pull requests that update a dependency file label Jan 12, 2022
@artonge artonge self-assigned this Jan 12, 2022
@artonge
Copy link
Contributor Author

artonge commented Jan 12, 2022

Server's PR is green, let's merge it !

@artonge artonge requested review from a team, PVince81, skjnldsv and szaimen and removed request for a team January 12, 2022 16:22
@skjnldsv skjnldsv merged commit 02086f8 into master Jan 14, 2022
@skjnldsv skjnldsv deleted the deps/unpin_webpack branch January 14, 2022 08:23
@skjnldsv
Copy link
Contributor

Let's release a new (patch?) version before I release a new major for #274

@skjnldsv
Copy link
Contributor

done

@skjnldsv
Copy link
Contributor

You forgot to unpin the webpack peer dep

@skjnldsv skjnldsv mentioned this pull request Jan 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants