Add INI Parser library#149
Conversation
|
Hi @Hammad-hab , there are a couple of issues I can observe:
|
|
Ok, I'll take a look and resolve these issues |
sstadick
left a comment
There was a problem hiding this comment.
Note that you can test this with rattler-build build -c https://conda.modular.com/max -c conda-forge --skip-existing=all -r ./recipe.yaml after installing rattler-build locally.
| then: | ||
| - mojo test test_parser.mojo | ||
| files: | ||
| recipe: |
There was a problem hiding this comment.
suggestion: change recipe to source, and it will use the test files in your source repo. Double check that test_parser.mojo is at the top level. That way you don't have to include test files in this repo.
There was a problem hiding this comment.
Sorry I didn't catch this on the first look!
|
Hi @yetalit. |
|
Hi @Hammad-hab , following error occurred: |
|
Sorry about that. I fixed the recipe.yaml. |
Checklist
recipe.yamlfile specifies which version(s) of MAX is compatible with my project (see here for an example). If not, my package is compatible with both 24.5 and 24.6.0(for new packages, or if the version changed).