-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
add ipydrawio 1.0.1 #14528
add ipydrawio 1.0.1 #14528
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Hm, don't know what to do about the long windows paths... |
Going to try working the path length stuff upstream... basically impossible to patch from the sdist alone... |
I shaved about 30 characters off... some of the paths. We'll see how it goes! |
@conda-forge/staged-recipes @conda-forge/help-python ready for review! |
|
||
extra: | ||
recipe-maintainers: | ||
- bollwyvl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- bollwyvl | |
- bollwyvl | |
feedstock-name: ipydrawio |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
recipes/ipydrawio/meta.yaml
Outdated
{% set build_number = 0 %} | ||
|
||
package: | ||
name: {{ name }}-build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you do a s/{{ name }}/ipydrawio/g
for this file? It's hurting readability.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the speedy review! Have updated from comments... |
What's the reason for having one recipe instead of 2 for two different packages? |
It's to avoid the heavy dependencies (node) on the pdf export side. They are both cut from the same tag... in an ideal world they would be the same (python) package, but I tire of |
Thanks! |
Checklist
Apache-2.0
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).Notes: