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: zod path in fork #51

Merged
merged 4 commits into from
Jul 23, 2024
Merged

Conversation

cau777
Copy link
Contributor

@cau777 cau777 commented Jul 23, 2024

Task: https://www.notion.so/Fix-instanceof-on-zod-9c21631b782043fdb67a42e8263a40dc?pvs=4

This PR removes the zod patch and uses a forked version instead.

npm publish --dry-run says this should be deployable:

npm publish --dry-run
npm WARN publish npm auto-corrected some errors in your package.json when publishing.  Please run "npm pkg fix" to address these errors.
npm WARN publish errors corrected:
npm WARN publish "repository.url" was normalized to "git+https://github.com/tianhuil/ts-mongo.git"
npm notice 
npm notice 📦  @tianhuil/[email protected]
npm notice === Tarball Contents === 
npm notice 11.4kB LICENSE.md                                     
npm notice 3.8kB  README.md                                      
npm notice 2.2kB  lib/cjs/collection.d.ts                        
[...]               
npm notice 2.4kB  package.json                                   
npm notice === Tarball Details === 
npm notice name:          @tianhuil/ts-mongo                      
npm notice version:       2.4.0                                   
npm notice filename:      tianhuil-ts-mongo-2.4.0.tgz             
npm notice package size:  35.4 kB                                 
npm notice unpacked size: 236.7 kB                                
npm notice shasum:        df19169d912bf0288e88b952f1a7adc76c137113
npm notice integrity:     sha512-koD6pyKdv+Frs[...]w3+nqP6ltwUYA==
npm notice total files:   203                                     
npm notice 
npm notice Publishing to https://registry.npmjs.org/ with tag latest and public access (dry-run)
+ @tianhuil/[email protected]

@tianhuil tianhuil merged commit 2b4e796 into tianhuil:main Jul 23, 2024
1 check passed
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