Skip to content

CI: Setup repo for CI#2

Merged
joncinque merged 3 commits intosolana-program:mainfrom
joncinque:ci
Dec 17, 2024
Merged

CI: Setup repo for CI#2
joncinque merged 3 commits intosolana-program:mainfrom
joncinque:ci

Conversation

@joncinque
Copy link
Copy Markdown
Contributor

Problem

The associated-token-account program, interface, and tests are in the repo, but they aren't properly exercised through CI.

Summary of changes

Get the whole repo setup! Contains the following changes:

  • move program-test into program/tests for simplicity
  • remove unneeded use_latest_* functions in program-test
  • add CI for build / lint / format / test the program and interface
  • add rust-specific scripts and re-use them
  • update maintainers to Anza

NOTE: The interface crate is still called client, but since it functions as more of an interface crate, I moved it into interface. The crate rename can happen later.

@joncinque
Copy link
Copy Markdown
Contributor Author

@joncinque joncinque marked this pull request as ready for review December 16, 2024 20:48
@joncinque joncinque requested a review from febo December 16, 2024 20:48
Copy link
Copy Markdown

@febo febo left a comment

Choose a reason for hiding this comment

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

Looks great!

#### Problem

The associated-token-account program, interface, and tests are in the
repo, but they aren't properly exercised through CI.

#### Summary of changes

Get the whole repo setup! Contains the following changes:

* move `program-test` into `program/tests` for simplicity
* remove unneeded `use_latest_*` functions in program-test
* add CI for build / lint / format / test the program and interface
* add rust-specific scripts and re-use them
* update maintainers to Anza

NOTE: The interface crate is still called `client`, but since it
functions as more of an interface crate, I moved it into `interface`.
The crate rename can happen later.
@joncinque joncinque merged commit e115a7b into solana-program:main Dec 17, 2024
@joncinque joncinque deleted the ci branch December 17, 2024 21:51
Copy link
Copy Markdown

@agca52 agca52 left a comment

Choose a reason for hiding this comment

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

😀gelişim için destek

grod220 pushed a commit that referenced this pull request Mar 2, 2026
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.

3 participants