Skip to content

chore: deploy test contracts with cargo-near#157

Merged
karim-en merged 4 commits into
masterfrom
cargo-near
Feb 26, 2025
Merged

chore: deploy test contracts with cargo-near#157
karim-en merged 4 commits into
masterfrom
cargo-near

Conversation

@karim-en
Copy link
Copy Markdown
Collaborator

No description provided.

@karim-en karim-en requested review from frolvanya and r-near February 26, 2025 02:00
cargo_near_build::camino::Utf8PathBuf::from_str(path.to_str().unwrap())
.expect("camino PathBuf from str"),
),
no_abi: true, // TODO remove this flag when we fix ABI generation
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are there many errors right now? I thought that near-plugins were fixed to support abi

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Looks like not all issues are fixed.
@r-near can you please verify and fix that?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Happy to take a look - what's the issue exactly? The ABI is not being generated correctly? Or something else?

Copy link
Copy Markdown
Contributor

@frolvanya frolvanya Feb 26, 2025

Choose a reason for hiding this comment

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

probably there're abi issue during the compilation, you need to use #[near(serializers = [borsh, json])] to fix it

P.S: you can use this as a reference

@karim-en karim-en merged commit 81d9f6f into master Feb 26, 2025
@karim-en karim-en deleted the cargo-near branch February 26, 2025 02:05
This was referenced Mar 27, 2026
@github-actions github-actions Bot mentioned this pull request Apr 7, 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