Mark Farragher
Mark Farragher
I have a bit of spare time the next few days so I will give it a try and see how it goes. 🤞
I'm a novice with vector graphics and did tinkering with Inkscape so I'm pleased with how this is rendering 😄 ```svg foobar2000 icon ``` Before doing a PR is that...
Inkscape. I had tried methods like Union on the paths but not sure how to combine them. 🙂
Thanks I've made a PR - I've mentioned this issue in there so it's linked above.
@hursh-desai @macdrifter was this issue from the vault's dirpath not being a pathlib.Path object? (like with @daneah comment) If that's the case, this wouldn't be a bug as the library...
Use the dev branch for code that addresses this. I raise an exception for dirpath values that aren't pathlib.Path type since this commit: https://github.com/mfarragher/obsidiantools/pull/37#issuecomment-3006445012
Do a build of the package from the `dev` branch. Fix merged in a PR (taking the commit from this one): https://github.com/mfarragher/obsidiantools/pull/47
My current environment is using Python 3.12. The tests ran successfully locally for that and for 3.9, 3.10 & 3.11 in the CI/CD run.
Also making a note here that `tags` is part of the [Properties](https://help.obsidian.md/properties) feature: > For the name, you can choose anything you like. Obsidian provides several default properties: `tags`, `cssclasses`,...
These are a few API changes that I think can enable this: - Let class properties have setters - Allow `gather()` to be called without `connect()` I can see how...