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

[token-client] Remove proof-program feature in token-client #5684

Conversation

samkim-crypto
Copy link
Contributor

@samkim-crypto samkim-crypto commented Oct 27, 2023

Problem

The proof-program feature in token-client is not really used for anything and contains outdated code that was not removed when the other functions were updated.

Summary of changes

Remove the proof-program feature and the containing code.

@samkim-crypto samkim-crypto added the WIP Work in progress label Oct 27, 2023
@samkim-crypto samkim-crypto force-pushed the token-client/remove-proof-program-feature branch from 221d37d to e299bb9 Compare October 27, 2023 18:59
@samkim-crypto samkim-crypto removed the WIP Work in progress label Oct 27, 2023
joncinque
joncinque previously approved these changes Oct 27, 2023
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.

Looks good!

spl-token-2022 = { version = "0.9", path="../program-2022" }
spl-token-metadata-interface = { version = "0.2", path="../../token-metadata/interface" }
spl-transfer-hook-interface = { version = "0.3", path="../transfer-hook/interface" }
spl-associated-token-account = { version = "2.0", path = "../../associated-token-account/program", features = [
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like a bunch of unrelated formatting got in too, can you revert those?

@samkim-crypto samkim-crypto force-pushed the token-client/remove-proof-program-feature branch from e299bb9 to 4696e32 Compare October 27, 2023 22:12
@mergify mergify bot dismissed joncinque’s stale review October 27, 2023 22:13

Pull request has been modified.

@samkim-crypto samkim-crypto force-pushed the token-client/remove-proof-program-feature branch from 4696e32 to 5734ed1 Compare October 27, 2023 22:14
@samkim-crypto samkim-crypto merged commit 6c396cc into solana-labs:master Oct 28, 2023
31 checks passed
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