This repository has been archived by the owner on May 14, 2024. It is now read-only.
[accounts] Release #1631
Merged
[accounts] Release #1631
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This allows us to perform test deployments of any of the supported apps. Refs: - https://bash.cyberciti.biz/guide/The_case_statement - https://stackoverflow.com/questions/2172352/in-bash-how-can-i-check-if-a-string-begins-with-some-value
The version field is still required, but we can additionally add a private for both semantic signalling and preventing this from being published to npm.
> Compiling using Next.js compiler instead of Babel is enabled by default since v12 https://nextjs.org/docs/architecture/nextjs-compiler
- The star in package.json matches any version https://docs.npmjs.com/cli/v10/configuring-npm/package-json - VSCode is still giving errors
This is to prevent Next from forcefully overwriting the "moduleResolution" in tsconfig.json to "node". If it is "node", then VS code cannot find the imports we do from @repo/ui. This was a bug in Next which was fixed in this PR: vercel/next.js#51957 To upgrade, I used: yarn upgrade next@^13.5 The latest Next.js is 14, but I left it at the last release in the 13 series. Will deal with the upgrade to 14 later in a different PR.
This was because with 13.5 I was getting an error coming from emotion:
Unhandled Runtime Error
Error: Component selectors can only be used in conjunction with
@emotion/babel-plugin, the swc Emotion plugin, or another Emotion-aware compiler
transform.
Call Stack
- handleInterpolation
../node_modules/@emotion/serialize/dist/emotion-serialize.browser.esm.js (84:0)
- serializeStyles
../node_modules/@emotion/serialize/dist/emotion-serialize.browser.esm.js (264:0)
eslint 8 already supports both formats, and will start defaulting to the new one in 9. https://eslint.org/blog/2022/08/new-config-system-part-2/
VSCode still gives an error We had to set the type to module to get the `import` to work. Setting .mjs didn't work (eslint didn't recognize the file). Setting the type to module should be (based on my current understanding) not affect us negatively since we are exporting a TypeScript library not a bundled one, and all the other source files apart from this are intended to be `*.ts`. Refs: - https://jeremyrichardson.dev/blog/saga_of_learning_exports_property_in_packagejson - eslint/eslint#17021
The flat config requires enabling a flag in the VS code plugin, it in not
enabled by default.
"eslint.experimental.useFlatConfig": true
This is an unnecessary expectation from all developers on the project, so we for
now live with the old format.
README tweaks
This used to be an issue back then while downloading some packages, or so I've heard, let us see if it is still one.
eslint already ignores node_modules and folders beginning with a dot https://eslint.org/docs/latest/use/configure/ignore
We don't need to set `productionBrowserSourceMaps` because the Sentry Webpack plugin always assumes that to be true. - @sentry/nextjs withSentryConfig enables productionBrowserSourceMaps, in the end, assets sourcemap is open to the public vercel/next.js#32920 - @sentry/nextjs will generate and upload source maps automatically https://docs.sentry.io/platforms/javascript/guides/nextjs/sourcemaps/
I'm not sure if this was intentional, but it looks like the initialization of `pkg` was accidentally deleted in e3019b6.
Deploying with
|
| Latest commit: |
dd8f261
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://899d8d89.bada-frame.pages.dev |
| Branch Preview URL: | https://accounts-preview.bada-frame.pages.dev |
mnvr
approved these changes
Feb 18, 2024
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Test Plan