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

Hurl parser rust #516

Closed
wants to merge 8 commits into from
Closed

Hurl parser rust #516

wants to merge 8 commits into from

Conversation

WrathOP
Copy link
Contributor

@WrathOP WrathOP commented Dec 26, 2024

Need documentation

PR Description

hurl parser rust initial working using flutter_rust_bridge library

Most of the files are autogenerated.

Related Issues

Checklist

  • I have gone through the contributing guide
  • I have updated my branch and synced it with project main branch before making this PR
  • I am using the latest Flutter stable branch (run flutter upgrade and verify)
  • I have run the tests (flutter test) and all tests are passing

Added/updated tests?

We encourage you to add relevant test cases.

  • Yes
  • No, and this is why: please replace this line with details on why tests have not been included

OS on which you have developed and tested the feature?

  • Windows
  • macOS
  • Linux

- Need to implement models
- Converting model from json
@WrathOP
Copy link
Contributor Author

WrathOP commented Dec 26, 2024

@ashitaprasad I tried this as a POC and its working we are getting a nice json from the parser and I think we can go with this. If you could have a look.

and yes the for flutter tests you might need to build the rust project

cd rust
cargo build --release

@ashitaprasad
Copy link
Member

@WrathOP is this draft ready to review?

@WrathOP
Copy link
Contributor Author

WrathOP commented Dec 28, 2024

@WrathOP is this draft ready to review?

No It's not just wanted to ask if you are fine with the approach so that I can make the models.

@WrathOP WrathOP marked this pull request as ready for review December 29, 2024 09:47
@ashitaprasad
Copy link
Member

@WrathOP kindly rename the package from hurl_parser_rust to simply hurl everywhere.

@ashitaprasad
Copy link
Member

@WrathOP Provide a proper writeup detailing:

  • What is flutter_rust_bride
  • How does it work
  • Step by step instructions on how you created this package and integrated it with API Dash
  • Also, provide step by step instructions on how to run it.
    Another question is - Will this package work for android and iOS builds? have you tested it in iOS & Android simulators?

@ashitaprasad
Copy link
Member

Closing comments in #553

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.

Importing Requests from hurl file
2 participants