-
Notifications
You must be signed in to change notification settings - Fork 28k
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
adding with-axiom example #38300
adding with-axiom example #38300
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's convert to TypeScript. See: https://github.com/vercel/next.js/blob/canary/contributing.md#adding-examples
I have updated the PR now to be in TS, but pnpm is not working for me, it fails with GYP error. can we let it run through the CI? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's also add a tsconfig.json
(running next dev
will generate one)
Co-authored-by: Balázs Orbán <info@balazsorban.com>
added. (thanks for the tip, saved my time :D) |
@balazsorban44 is there something that needs to be updated? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for the PR! :)
37b51f1
## Documentation / Examples - [ X ] Make sure the linting passes by running `pnpm lint` - [ X ] The examples guidelines are followed from [our contributing doc](https://github.com/vercel/next.js/blob/canary/contributing.md#adding-examples) This PR adds an example for using next.js with axiom. Co-authored-by: Balázs Orbán <info@balazsorban.com> Co-authored-by: Steven Tey <stevensteel97@gmail.com>
Documentation / Examples
pnpm lint
This PR adds an example for using next.js with axiom.