Skip to content

[jnigen] Use String instead of Uri everywhere #1371

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

Closed
HosseinYousefi opened this issue Jul 23, 2024 · 2 comments
Closed

[jnigen] Use String instead of Uri everywhere #1371

HosseinYousefi opened this issue Jul 23, 2024 · 2 comments

Comments

@HosseinYousefi
Copy link
Member

No description provided.

@dcharkes
Copy link
Collaborator

dcharkes commented Sep 4, 2024

@HosseinYousefi Please contribute to the discussion #1506 if you care. 👌 Then we can align the packages. (Especially, when we start using the Dart instead of YAML config and we get a hook for running code-gens. Then we don't want to be converting between File/Uri/String.)

@HosseinYousefi HosseinYousefi moved this to Backlog in JNIgen tracker Sep 9, 2024
@HosseinYousefi HosseinYousefi changed the title [jnigen] Use Uri instead of String everywhere [jnigen] Use String instead of Uri everywhere Oct 8, 2024
@HosseinYousefi HosseinYousefi added this to the JNI / JNIgen 0.13.0 milestone Oct 8, 2024
@HosseinYousefi HosseinYousefi moved this from Backlog to In Progress in JNIgen tracker Oct 8, 2024
@HosseinYousefi HosseinYousefi self-assigned this Oct 8, 2024
@HosseinYousefi
Copy link
Member Author

HosseinYousefi commented Oct 8, 2024

I think using strings is simplest in the move to a dart config (instead of yaml). Will do this after removing the yaml config because package:config uses Uris.

@HosseinYousefi HosseinYousefi moved this from In Progress to Todo in JNIgen tracker Oct 8, 2024
@HosseinYousefi HosseinYousefi closed this as not planned Won't fix, can't repro, duplicate, stale Nov 27, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in JNIgen tracker Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

2 participants