-
Notifications
You must be signed in to change notification settings - Fork 254
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
Version Packages #842
Version Packages #842
Conversation
ESLint Summary View Full Report
Report generated by eslint-plus-action |
I think that fixes it. |
Okay, great. Thanks @luisherranz 🙂 |
The Let's see if that fixes the error. |
Uhm, weird, I didn't fix anything... 🤔 |
Mmm, if they are passing locally, could be something related to npm? I remember we have had issues in the past because we use npm 7 and GitHub actions use npm 6. |
Actually it's failing with node 16 and npm 7. |
OK, I didn't try with those versions before. 🙈 Not it fails in my local env with a more related error: |
It doesn't seem specific to this particular PR because I just pushed to this other one and it also failed: #836 |
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.
Releases
@frontity/components@1.8.0
Minor Changes
2f988427
#826 Thanks @michalczaplinski! - Make the Image component from@frontity/components
AMP-aware and remove the Intersection Observer fallback which used to lazily load images in the absence of native lazy loading.@frontity/core@1.14.2
Patch Changes
a154a9cb
#840 Thanks @luisherranz! - Fix Webpack/TS-Node excludes not working on Windows.63154b4f
#837 Thanks @luisherranz! - Fix transpilation offrontity.config.js
files in external packages.