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

solana: Add build, test, and lint Makefile targets to top-level dir #299

Closed

Conversation

johnsaigle
Copy link
Collaborator

@johnsaigle johnsaigle commented Mar 12, 2024

Adds Makefile targets for common Solana tasks in the top-level directory. This allows the test and build pipeline to be run from the main directory for both the EVM and Solana implementations.

This also helps to capture the Solana commands that are configured in the .github/workflows/solana.yml file. In case this codebase is merged or adapted into the Wormhole monorepo, the GitHub workflow may be removed for modified. Therefore it is beneficial to track the expected test configurations here.

See also: #285. Part of the motivation for adding these Makefile targets is to configure additional clippy rules that can be run locally in addition to CI runs.

@johnsaigle johnsaigle changed the title solana: Add build, test, and lint targets to top-level dir solana: Add build, test, and lint Makefile targets to top-level dir Mar 12, 2024
@johnsaigle johnsaigle force-pushed the solana/tests-in-make branch 2 times, most recently from bf279a9 to 31a4153 Compare March 18, 2024 16:35
@nik-suri nik-suri added the solana Change to Solana programs label Mar 25, 2024
@nik-suri nik-suri requested a review from kcsongor May 8, 2024 17:46
Adds Makefile targets for common Solana tasks in the top-level
directory. This allows the test and build pipeline to be run from the
main directory for both the EVM and Solana implementations.

This also helps to capture the Solana commands that are configured in
the .github/workflows/solana.yml file. In case this codebase is merged
or adapted into the Wormhole monorepo, the GitHub workflow may be
removed for modified. Therefore it is beneficial to track the
expected test configurations here.
@nvsriram
Copy link
Collaborator

Changes merged through #502

@nvsriram nvsriram closed this Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 solana Change to Solana programs tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants