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

Require Rescript-schema in top-level package.json #328

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JasoonS
Copy link
Contributor

@JasoonS JasoonS commented Nov 8, 2024

image

I can't explain why I had this issue all of a sudden on the latest version, but I did... (2 weeks ago I didn't have this issue, only had this issue: JasoonS/basic-envio-ens-example@19144a9 - but that has been fixed 🙏 )

@JasoonS JasoonS requested a review from DZakh November 8, 2024 06:49
@DZakh
Copy link
Member

DZakh commented Nov 8, 2024

It should work without rescript-schema 🤔 I'll investigate

@DZakh
Copy link
Member

DZakh commented Nov 8, 2024

Could you share the reproduction steps? I don't see any error when I init a ReScript template.

@JasoonS
Copy link
Contributor Author

JasoonS commented Dec 19, 2024

Here is the reproduction (sorry for the delay):
Basically do a rescript indexer on the 0x283Af0B28c62C092C9727F1Ee09c02CA627EB7F5 contract on Mainnet Ethereum.

image

@JasoonS
Copy link
Contributor Author

JasoonS commented Dec 19, 2024

I did this again and now I needed to install pino, pino-pretty and ethers 🤔
Must be something with the way the pnpm does linking or something.

@DZakh
Copy link
Member

DZakh commented Dec 19, 2024

Could you paste the shell content as text

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