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

Cli: Don't regenerate idl in read_all_programs() #2332

Merged
merged 2 commits into from
Dec 23, 2022

Conversation

ckamm
Copy link
Contributor

@ckamm ckamm commented Dec 21, 2022

Resolving a TODO and making idl tooling work more nicely with manual adjustments to the on-disk idl.

@vercel
Copy link

vercel bot commented Dec 21, 2022

@ckamm is attempting to deploy a commit to the coral-xyz Team on Vercel.

A member of the Team first needs to authorize it.

@Henry-E
Copy link
Contributor

Henry-E commented Dec 22, 2022

Looks good, should be a small enough change to leave out of the changelog but whatever you prefer. I'm happy to merge as is.

@Henry-E Henry-E merged commit a97d04a into coral-xyz:master Dec 23, 2022
dovahcrow pushed a commit to dovahcrow/anchor that referenced this pull request Jan 13, 2023
stegaBOB added a commit to staratlasmeta/anchor that referenced this pull request Aug 7, 2023
This commit optimizes the method of reading all programs in `config.rs` file. It replaces the old method of parsing the source with reading directly from the target IDL directory. This enables us to run `anchor localnet` without needing the entire program source files. Backport of coral-xyz#2332
github-merge-queue bot pushed a commit to staratlasmeta/anchor that referenced this pull request Aug 7, 2023
This commit optimizes the method of reading all programs in `config.rs` file. It replaces the old method of parsing the source with reading directly from the target IDL directory. This enables us to run `anchor localnet` without needing the entire program source files. Backport of coral-xyz#2332
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants