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

feat: Remove deprecated return value on .js and .css methods #117

Merged
merged 1 commit into from
Jul 19, 2020

Conversation

trygve-lie
Copy link
Contributor

BREAKING CHANGE: In version 4 of podium it became possible to set multiple assets to a podlet through its .js() and .css() methods. This did make it impossible to let these methods return a reasonable value, but to keep a backwards compatibility with version 3 of Podium, the first item passed in was returned for compatibility. This removes this compatibility with version 3 of Podium.

This resolves podium-lib/issues#23

BREAKING CHANGE: In version 4 of podium it became possible to set multiple assets to a podlet through its `.js()` and `.css()` methods. This did make it impossible to let these methods return a resonable value, but to keep a backwards compabillity with version 3 of Podium, the first item passed in was returned for compabillity. This removes this compaillity with version 3 of Podium.
@trygve-lie trygve-lie merged commit ddc13a9 into next Jul 19, 2020
@trygve-lie trygve-lie deleted the return branch July 19, 2020 19:23
@github-actions
Copy link

🎉 This PR is included in version 5.0.0-next.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@trygve-lie trygve-lie added the v5 Version 5 label Jul 19, 2020
Copy link

🎉 This PR is included in version 5.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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants