Skip to content

feat(cli): add CLI command idl to generate IDL from source crate#589

Merged
vobradovich merged 15 commits into
masterfrom
vo/cli-idl-gen
Oct 23, 2024
Merged

feat(cli): add CLI command idl to generate IDL from source crate#589
vobradovich merged 15 commits into
masterfrom
vo/cli-idl-gen

Conversation

@vobradovich
Copy link
Copy Markdown
Member

@vobradovich vobradovich commented Oct 17, 2024

Usages

  • with current dir
cargo sails idl
  • with manifest-path
cargo sails idl --manifest-path E:\git\sails-test\app\Cargo.toml

@vobradovich vobradovich self-assigned this Oct 17, 2024
@DennisInSky DennisInSky changed the base branch from master to vo/idl-gen-meta October 17, 2024 14:31
Copy link
Copy Markdown
Contributor

@DennisInSky DennisInSky left a comment

Choose a reason for hiding this comment

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

Some CI test would be nice:

  • we have one which generates a program
  • then you can use generated program for obtaining its idl
  • then maybe compare obtained idl with the one generated during program test

Comment thread rs/cli/src/idlgen.rs Outdated
Comment thread rs/cli/src/idlgen.rs
Comment thread rs/cli/src/idlgen.rs Outdated
Comment thread rs/cli/src/idlgen.rs Outdated
Comment thread rs/cli/src/idlgen.rs Outdated
Comment thread rs/cli/src/idlgen.rs
Comment thread rs/cli/src/idlgen.rs Outdated
Comment thread rs/cli/src/idlgen.rs Outdated
Base automatically changed from vo/idl-gen-meta to master October 21, 2024 11:26
@vobradovich vobradovich changed the title feat(cli): add CLI command idl-gen to generate IDL from source crate feat(cli): add CLI command idl to generate IDL from source crate Oct 22, 2024
Comment thread Cargo.toml Outdated
Comment thread .github/workflows/ci.yml
Comment thread rs/cli/src/idlgen.rs Outdated
Comment thread rs/cli/src/idlgen.rs Outdated
@vobradovich vobradovich merged commit 418c351 into master Oct 23, 2024
@vobradovich vobradovich deleted the vo/cli-idl-gen branch October 23, 2024 14:42
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