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

fix: more type checking errors #87

Merged
merged 1 commit into from
Feb 5, 2024
Merged

Conversation

deer
Copy link
Contributor

@deer deer commented Feb 5, 2024

closes #85

@miguelrk, as detailed in the comment in the issue, the approach here runs and makes the type checker happy.

There were a few miscellaneous fixes as well, in order to get the check:types task to even run... It now finally tells me:

Found 454 errors.

I guess there's some more work to do 😅

@miguelrk
Copy link
Contributor

miguelrk commented Feb 5, 2024

That's great! Thank you!

I'll merge this one for now, I opened #88 to track progress for the remaining errors

@miguelrk miguelrk merged commit 3dd7362 into netzo:main Feb 5, 2024
@deer deer deleted the fix_more_errors branch February 5, 2024 13:28
miguelrk added a commit that referenced this pull request Feb 6, 2024
…c-analyzable dynamic import

Chanes introduced in #87 are somehow leading to the following error in deployment "The deployment failed: UNCAUGHT_EXCEPTION TypeError: module not found: 'file:///src/fresh.gen.ts' at async Object.start (https://deno.land/x/netzo@0.4.21/core/mod.ts:113:23)
". This is an attempt to fix it.
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.

[core] improve dynamic import in lib/mod.ts
2 participants