Skip to content

p-token: Add scripts #27

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

Merged
merged 9 commits into from
Mar 10, 2025
Merged

p-token: Add scripts #27

merged 9 commits into from
Mar 10, 2025

Conversation

febo
Copy link
Contributor

@febo febo commented Feb 18, 2025

Problem

Currently it is not possible to build and test p-token program using pnpm scripts.

Solution

Add p-token:build and p-token:test to package.json to build and test the program.

Starting from a fresh clone:

pnpm install

to install pnpm dependencies. Then we can use:

pnpm p-token:build

to build p-token binary – it will be located at ./target/deploy/pinocchio_token_program.so. You can also use:

pnpm p-token:test

to run p-token tests.

@febo febo force-pushed the febo/spellcheck branch from 9b7b2f5 to d443730 Compare March 7, 2025 22:30
@febo febo force-pushed the febo/p-token-scripts branch from 76bd300 to 64bd48f Compare March 7, 2025 22:49
@febo febo marked this pull request as ready for review March 7, 2025 22:54
@febo febo requested a review from joncinque March 7, 2025 22:54
Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

All of the renames probably belong in #24, but no big deal, it'll all land anyway, looks good!

@febo febo force-pushed the febo/spellcheck branch from d443730 to 60115db Compare March 8, 2025 02:58
@febo febo force-pushed the febo/p-token-scripts branch from 64bd48f to 7981197 Compare March 8, 2025 03:00
Base automatically changed from febo/spellcheck to main March 10, 2025 13:58
@febo febo force-pushed the febo/p-token-scripts branch from 7981197 to bc765ed Compare March 10, 2025 14:12
@febo febo merged commit 08540e2 into main Mar 10, 2025
11 checks passed
@febo febo deleted the febo/p-token-scripts branch March 10, 2025 14:24
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