Skip to content

CI: Bring repo up to date#3

Merged
joncinque merged 7 commits into
solana-program:mainfrom
joncinque:ci
Mar 8, 2025
Merged

CI: Bring repo up to date#3
joncinque merged 7 commits into
solana-program:mainfrom
joncinque:ci

Conversation

@joncinque
Copy link
Copy Markdown
Contributor

Problem

The token-metadata repo exists, but isn't setup with CI or anything.

Summary of changes

Similar to solana-program/token-group#2, but for a different repo.

#### Problem

The token-metadata repo exists, but isn't setup with CI or anything.

#### Summary of changes

Similar to solana-program/token-group#2, but for
a different repo.
@joncinque joncinque requested a review from febo March 7, 2025 13:40
@joncinque
Copy link
Copy Markdown
Contributor Author

Comment thread interface/Cargo.toml Outdated
Comment thread program/Cargo.toml
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 good - just a couple of tiny nits.

@joncinque joncinque merged commit 1a52aa1 into solana-program:main Mar 8, 2025
@joncinque joncinque deleted the ci branch March 8, 2025 00:13
joncinque added a commit to joncinque/token-metadata that referenced this pull request Mar 8, 2025
#### Problem

We made spl-token-metadata-example purely a cdylib in solana-program#3, but the tests
were using it as a library.

#### Summary of changes

That code wasn't being used anyway, so force the test to use the built
BPF program.

Also clean up some other stuff.
joncinque added a commit that referenced this pull request Mar 14, 2025
* CI: Fix lint and test program jobs

#### Problem

We made spl-token-metadata-example purely a cdylib in #3, but the tests
were using it as a library.

#### Summary of changes

That code wasn't being used anyway, so force the test to use the built
BPF program.

Also clean up some other stuff.

* Rustfmt
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