-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
spl: change "to" to "from" in token::burn #1080
Conversation
Can you add a changelog entry in a |
6df6e6a
to
658d314
Compare
09cc6b9
to
44c129d
Compare
fix makefile
I fixed the makefile, now it should pass the tests also 🙏 @armaniferrante can you link the issue #161 to the PR? |
@@ -3,47 +3,56 @@ build-cli: | |||
cargo build -p anchor-cli --release | |||
cp target/release/anchor cli/npm-package/anchor | |||
|
|||
.PHONY: build-tests-bpf-% |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are makefile changes needed? Seems like they shouldn't be.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On the current makefile rules build-example-bpf-all do not work because basic-0 to basic-4 are not located on "examples/" but in "examples/tutorial" (and basic-5 do not exist anymore)
Also all other examples are not located on "examples/" but on "tests/".
Since tests and tutorial is now split in two different folder I separate them in 2 different rule: "build-tests-bpf-all" and "build-example-bpf-all".
I added the rule "build-all" to build tests and tutorials.
hey @inso- are you interested in finishing this? if yes, pls undo the makefile changes. We have a separate issue for that |
closed by #1729 |
This is quite confusing and a suggestion to change this already have been approved.
#161