chore: fix duplicate package.json field, update gitignore and docs#66
chore: fix duplicate package.json field, update gitignore and docs#66brawlaphant wants to merge 1 commit intoregen-network:mainfrom
Conversation
- Remove duplicate "type": "module" field from package.json - Add Python and Rust/CosmWasm build artifact patterns to .gitignore - Update README.md license section with Apache 2.0 and CC BY-SA 4.0 - Add verify:m010:datasets script row and M012/M013/M015 self-test examples to docs/DEV_SETUP.md Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request focuses on repository maintenance and developer experience improvements. It addresses minor configuration issues, enhances the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request updates the .gitignore file to include Python and Rust/CosmWasm build artifacts, clarifies licensing details in README.md, and enhances docs/DEV_SETUP.md with a new verification script for M010 datasets and instructions for running self-tests for M012, M013, and M015 mechanisms. A review comment highlights a duplicate 'type': 'module' field in package.json that should be removed to maintain valid JSON.
| "private": true, | ||
| "type": "module", | ||
| "version": "0.0.0", | ||
| "type": "module", |
There was a problem hiding this comment.
There was a problem hiding this comment.
Thanks for flagging — this PR is the fix for that exact issue. The diff shows the duplicate "type": "module" being removed (the - line). After this PR merges, package.json will have exactly one "type": "module" field on line 4.
Summary
"type": "module"field (appeared on both line 4 and line 6)__pycache__/,*.pyc,*.pyo,.venv/,*.egg-info/,dist/,build/) and Rust/CosmWasm (target/,*.wasm) patterns in preparation for PR feat: cadCAD economic simulations for M012-M015 parameter validation #58 cadCAD simulation codeverify:m010:datasetsnpm script to the table; add M012, M013, M015 reference implementation self-test examplesTest plan
node scripts/verify.mjsstill passesnpm run verify:m010:datasetsruns correctlypackage.jsonparses without errors (node -e "require('./package.json')"orJSON.parse).gitignorepatterns exclude expected files🤖 Generated with Claude Code