Skip to content

Conversation

@alextrnnn
Copy link
Contributor

Changes in this pull request

Add a C binding for setting the base path, which will be used to show an example of loading ingredients from a folder.

Related to #1231

Checklist

  • This PR represents a single feature, fix, or change.
  • All applicable changes have been documented.
  • Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.

@alextrnnn alextrnnn self-assigned this Jul 16, 2025
@alextrnnn alextrnnn requested a review from gpeacock July 16, 2025 18:54
Copy link
Contributor

@ok-nick ok-nick left a comment

Choose a reason for hiding this comment

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

Few things, but looks good otherwise.

@alextrnnn alextrnnn merged commit 7a4a421 into main Jul 18, 2025
17 checks passed
@alextrnnn alextrnnn deleted the alextrnnn/resource-path branch July 18, 2025 22:37
@caiopensrc caiopensrc mentioned this pull request Jul 18, 2025
ok-nick pushed a commit that referenced this pull request Aug 11, 2025
* feat: Add set_resource_dir and binding, add_resource only writes to memory

* test: Add test for reading from ingredient folder using resource_dir

* refactor: Rename back to base path and remove breaking changes

* docs: Update documentation and warning

* chore: Fix test name

* fix: Make test consistent with others
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