Skip to content

Codestyle unification #187

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

Merged
merged 7 commits into from
Aug 28, 2023
Merged

Codestyle unification #187

merged 7 commits into from
Aug 28, 2023

Conversation

sladg
Copy link
Contributor

@sladg sladg commented Aug 25, 2023

What this does?

@changeset-bot
Copy link

changeset-bot bot commented Aug 25, 2023

⚠️ No Changeset found

Latest commit: 9b06bf6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

@khuezy khuezy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Everything is style/cosmetic except the image opt stream right? I'll test this in a bit then we can merge.

@khuezy khuezy self-requested a review August 25, 2023 16:10
Copy link
Contributor

@khuezy khuezy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there's a couple of uncommitted files

@vercel
Copy link

vercel bot commented Aug 27, 2023

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

Name Status Preview Comments Updated (UTC)
open-next ❌ Failed (Inspect) Aug 28, 2023 6:51pm

@khuezy
Copy link
Contributor

khuezy commented Aug 27, 2023

@sladg is there anything special I need to do to get the autoformat to work? I have eslint and prettier plugin installed in my VSCode, but when I save a file, it doesn't add the commas.
The eslint is picking up the rules but it's not applying the changes on save.

@sladg
Copy link
Contributor Author

sladg commented Aug 27, 2023

@khuezy disable prettier plugin. Only eslint is needed as prettier does not have configuration file in workspace so it's fighting with eslint most likely.

@khuezy
Copy link
Contributor

khuezy commented Aug 27, 2023

I think I have some weird VSCode settings in my global VSCode settings somewhere in my /Library/Application Support/Code/User. It might be better to create a .vscode/settings.json file to force the project to adhere to open-next's VSCode settings.

@khuezy
Copy link
Contributor

khuezy commented Aug 28, 2023

My VSCode broke... so I can't really test this PR smh...

@khuezy
Copy link
Contributor

khuezy commented Aug 28, 2023

I was using typescript 5.2.2 and the eslint lib didn't support it yet. The fix got deployed ~30m ago and everything is working after I reinstall to pick up new packages. See: typescript-eslint/typescript-eslint#7155 (comment)

We should upgrade to latest TS too

@khuezy
Copy link
Contributor

khuezy commented Aug 28, 2023

Do you mind adding .vscode/settings.json:

{
  "editor.codeActionsOnSave": {
    "source.fixAll.eslint": true
  },
}

That will autoformat to the style on save.

@khuezy
Copy link
Contributor

khuezy commented Aug 28, 2023

I also had to make some changes to server-adapter.ts. Let me know if you want me to push the changes up or not.

Copy link
Contributor

@khuezy khuezy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Thanks!

@khuezy khuezy merged commit d221fbc into opennextjs:main Aug 28, 2023
@sladg sladg deleted the feat/codestyle branch August 28, 2023 19:14
breningham pushed a commit to breningham/open-next that referenced this pull request Aug 29, 2023
* feat(eslint): configuration added for eslint and general styling
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.

Codestyle guidelines
2 participants