Skip to content

Conversation

@Raiden1411
Copy link
Collaborator

@Raiden1411 Raiden1411 commented May 8, 2023

Additional Information

  • I read the contributing guide
  • I added documentation related to the changes made.
  • I added or updated tests related to the changes made.

Your ENS/address:


PR-Codex overview

This PR moves configuration files into a dedicated dev folder and updates various file paths accordingly. It also makes minor changes to some scripts and adds new utility functions.

Detailed summary

  • Moves configuration files to dev folder
  • Updates file paths in various files accordingly
  • Adds new utility functions caller, filePath, and shell to dev/utils/utils.ts
  • Updates tsconfig.json and tsconfig.node.json to reference new file paths
  • Updates vitest.config.ts to reference new test file paths
  • Updates prepublishOnly.ts to reference the correct package.json location
  • Updates updateVersion.ts to reference the correct package.json and version.ts locations
  • Updates on-push-to-main.yml to include cwd field
  • Makes minor changes to changeset, lint, and format scripts to reference new configuration files

The following files were skipped due to too many changes: dev/utils/getCurrentLine.ts

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@changeset-bot
Copy link

changeset-bot bot commented May 8, 2023

⚠️ No Changeset found

Latest commit: 8e9b07d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@commit-lint
Copy link

commit-lint bot commented May 8, 2023

Chore

  • move configuration files into dedicated folder (f4beefb)
  • changeset (8e9b07d)

Contributors

Raiden1411

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

  • @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
  • @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
  • @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
  • @Commit-Lint merge disable will desactivate merge dependabot PR
  • @Commit-Lint review will approve dependabot PR
  • @Commit-Lint stop review will stop approve dependabot PR

@vercel
Copy link

vercel bot commented May 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
abitype ✅ Ready (Inspect) Visit Preview May 8, 2023 3:14pm

@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2023

Size Change: 0 B

Total Size: 46.5 kB

ℹ️ View Unchanged
Filename Size
dist/cjs/abi.js 123 B
dist/cjs/config.js 126 B
dist/cjs/errors.js 577 B
dist/cjs/human-readable/errors/abiItem.js 537 B
dist/cjs/human-readable/errors/abiParameter.js 804 B
dist/cjs/human-readable/errors/index.js 507 B
dist/cjs/human-readable/errors/signature.js 420 B
dist/cjs/human-readable/errors/splitParameters.js 404 B
dist/cjs/human-readable/errors/struct.js 338 B
dist/cjs/human-readable/index.js 543 B
dist/cjs/human-readable/parseAbi.js 329 B
dist/cjs/human-readable/parseAbiItem.js 416 B
dist/cjs/human-readable/parseAbiParameter.js 451 B
dist/cjs/human-readable/parseAbiParameters.js 501 B
dist/cjs/human-readable/runtime/cache.js 616 B
dist/cjs/human-readable/runtime/index.js 356 B
dist/cjs/human-readable/runtime/signatures.js 777 B
dist/cjs/human-readable/runtime/structs.js 999 B
dist/cjs/human-readable/runtime/utils.js 2.09 kB
dist/cjs/human-readable/types/index.js 125 B
dist/cjs/human-readable/types/signatures-test-d.js 438 B
dist/cjs/human-readable/types/signatures.js 127 B
dist/cjs/human-readable/types/structs.js 127 B
dist/cjs/human-readable/types/utils.js 124 B
dist/cjs/index.js 560 B
dist/cjs/narrow.js 169 B
dist/cjs/regex.js 362 B
dist/cjs/test/abis.js 5.93 kB
dist/cjs/test/human-readable.js 3.59 kB
dist/cjs/test/index.js 595 B
dist/cjs/types.js 124 B
dist/cjs/utils.js 124 B
dist/cjs/version.js 149 B
dist/cjs/zod/index.js 502 B
dist/cjs/zod/zod.js 1.28 kB
dist/esm/abi.js 62 B
dist/esm/config.js 65 B
dist/esm/errors.js 524 B
dist/esm/human-readable/errors/abiItem.js 454 B
dist/esm/human-readable/errors/abiParameter.js 674 B
dist/esm/human-readable/errors/index.js 251 B
dist/esm/human-readable/errors/signature.js 339 B
dist/esm/human-readable/errors/splitParameters.js 345 B
dist/esm/human-readable/errors/struct.js 282 B
dist/esm/human-readable/index.js 251 B
dist/esm/human-readable/parseAbi.js 257 B
dist/esm/human-readable/parseAbiItem.js 344 B
dist/esm/human-readable/parseAbiParameter.js 373 B
dist/esm/human-readable/parseAbiParameters.js 428 B
dist/esm/human-readable/runtime/cache.js 554 B
dist/esm/human-readable/runtime/index.js 167 B
dist/esm/human-readable/runtime/signatures.js 567 B
dist/esm/human-readable/runtime/structs.js 926 B
dist/esm/human-readable/runtime/utils.js 2.02 kB
dist/esm/human-readable/types/index.js 64 B
dist/esm/human-readable/types/signatures-test-d.js 365 B
dist/esm/human-readable/types/signatures.js 69 B
dist/esm/human-readable/types/structs.js 66 B
dist/esm/human-readable/types/utils.js 64 B
dist/esm/index.js 262 B
dist/esm/narrow.js 97 B
dist/esm/regex.js 284 B
dist/esm/test/abis.js 5.8 kB
dist/esm/test/human-readable.js 3.49 kB
dist/esm/test/index.js 296 B
dist/esm/types.js 64 B
dist/esm/utils.js 64 B
dist/esm/version.js 82 B
dist/esm/zod/index.js 221 B
dist/esm/zod/zod.js 1.13 kB

compressed-size-action

@codecov
Copy link

codecov bot commented May 8, 2023

Codecov Report

Merging #133 (8e9b07d) into main (508cd92) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #133   +/-   ##
=======================================
  Coverage   99.83%   99.83%           
=======================================
  Files          22       22           
  Lines        1792     1792           
  Branches      257      257           
=======================================
  Hits         1789     1789           
  Misses          3        3           

@Raiden1411 Raiden1411 requested a review from tmm May 8, 2023 15:18
@Raiden1411
Copy link
Collaborator Author

Will close this too. Reasons can be found here

@Raiden1411 Raiden1411 closed this May 9, 2023
@Raiden1411 Raiden1411 deleted the chore/configs branch May 9, 2023 18:13
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