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

Upgrade to remix 1.7.6 #725

Merged
merged 3 commits into from
Jan 3, 2023
Merged

Upgrade to remix 1.7.6 #725

merged 3 commits into from
Jan 3, 2023

Conversation

TrySound
Copy link
Member

https://github.com/remix-run/remix/releases/tag/remix%401.7.0 https://github.com/remix-run/remix/releases/tag/remix%401.7.4

du -ckd1 node_modules
-795648
+798524

Not much changes though I found since 1.7.5 build started to hang when run with turbo or recursive pnpm run.

Looks like in this commit some stream or connection not closed when build is finished.

remix-run/remix#4410

Let's see if not fixed in newer versions and then file an issue.

Code Review

  • hi @kof, I need you to do
    • detailed review (read every line)
    • test it on preview

Before requesting a review

  • made a self-review
  • added inline comments where things may be not obvious (the "why", not "what")

Before merging

  • tested locally and on preview environment (preview dev login: 5de6)
  • updated test cases document
  • added tests
  • if any new env variables are added, added them to .env.example and the designer/env-check.js if mandatory

https://github.com/remix-run/remix/releases/tag/remix%401.7.0
https://github.com/remix-run/remix/releases/tag/remix%401.7.4

du -ckd1 node_modules
-795648
+798524

Not much changes though I found since [1.7.5](https://github.com/remix-run/remix/releases/tag/remix%401.7.5) build started to hang when run with turbo or recursive pnpm run.

Looks like in this commit some stream or connection not closed when
build is finished.

remix-run/remix#4410

Let's see if not fixed in newer versions and then file an issue.
@TrySound TrySound requested a review from kof December 31, 2022 09:42
@vercel
Copy link

vercel bot commented Dec 31, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
webstudio-designer ✅ Ready (Inspect) Visit Preview Dec 31, 2022 at 9:53AM (UTC)

@TrySound
Copy link
Member Author

TrySound commented Dec 31, 2022

Getting following errors in dev
image

@kof
Copy link
Member

kof commented Jan 1, 2023

Did you mean the errors are coming with this update? That would mean we can't merge this ,right?

@TrySound
Copy link
Member Author

TrySound commented Jan 3, 2023

Maybe this was there and remix just started to show it. Do you know where these attributes may come from?

@kof
Copy link
Member

kof commented Jan 3, 2023

data-lt-installed doesn't exist anywhere, it is probably caused by Grammarly extension, try incognito mode

@kof
Copy link
Member

kof commented Jan 3, 2023

vercel/next.js#41816

@kof
Copy link
Member

kof commented Jan 3, 2023

I haven't tested, approving in assumption that you did :)

@TrySound
Copy link
Member Author

TrySound commented Jan 3, 2023

Great, catch. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants