Skip to content

feature: instructions builders convenience functions directory#299

Open
inspi-writer001 wants to merge 1 commit intogillsdk:masterfrom
inspi-writer001:builders-folder
Open

feature: instructions builders convenience functions directory#299
inspi-writer001 wants to merge 1 commit intogillsdk:masterfrom
inspi-writer001:builders-folder

Conversation

@inspi-writer001
Copy link

Problem

There currently is no folder for the builders convenience functions

Summary of Changes

Added a new folder builders in the packages/gill/src/ directory like so

packages/gill/src/
├── builders
│   └── index.ts
...

@changeset-bot
Copy link

changeset-bot bot commented Sep 23, 2025

⚠️ No Changeset found

Latest commit: c4cb663

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.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

@inspi-writer001 inspi-writer001 marked this pull request as ready for review September 24, 2025 16:33
Copy link
Collaborator

@tobeycodes tobeycodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just a file with a code comment

@inspi-writer001
Copy link
Author

inspi-writer001 commented Sep 30, 2025

This is just a file with a code comment

yeah, it's an index file in a builders/ folder
The aim is for other folks working on the builders to have a builders/ folder to work with.

@bundlemon
Copy link

bundlemon bot commented Oct 22, 2025

BundleMon

Files removed (3)
Status Path Size Limits
solana-pay/dist/index.browser.mjs
-1.72KB -
solana-pay/dist/index.native.mjs
-1.72KB -
solana-pay/dist/index.node.mjs
-1.72KB -
Files updated (8)
Status Path Size Limits
gill/dist/programs/token/index.node.mjs
4.09KB (-13B -0.31%) -
gill/dist/programs/index.node.mjs
9.83KB (-15B -0.15%) -
gill/dist/index.node.mjs
488B (-31B -5.97%) -
react/dist/index.browser.mjs
1.79KB (-337B -15.52%) -
react/dist/index.native.mjs
1.79KB (-337B -15.53%) -
react/dist/index.node.mjs
1.79KB (-337B -15.54%) -
gill/dist/index.browser.mjs
5.19KB (-352B -6.21%) -
gill/dist/index.native.mjs
5.19KB (-352B -6.22%) -
Unchanged files (5)
Status Path Size Limits
gill/dist/node/index.node.mjs
947B -
svelte/dist/index.browser.mjs
265B -
vue/dist/index.browser.mjs
265B -
svelte/dist/index.node.mjs
262B -
vue/dist/index.node.mjs
262B -

Total files change -6.89KB -17.68%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

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