-
-
Notifications
You must be signed in to change notification settings - Fork 312
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
Implement libraries.io support for pre-commit #345
Comments
have a look at this commit, i used the let me know if this is what you've imagined and if you have comments on it. |
looks like a good start! I imagine the version information would be useful as well -- is there a second mechanism for parsing |
the reason why i commented out the not sure if i understood you correct regarding consuming repositories: do you mean parsing the {
"https://github.com/pre-commit/mirrors-coffeelint": [
{
"id": "coffeelint",
"name": "coffeelint",
"entry": "coffeelint",
"language": "node",
"files": "\\.(js|coffee)$",
"args": [],
"additional_dependencies": ["[email protected]"],
"minimum_pre_commit_version": "0.7.0"
}
]
} would return the following in [
{
project_name: 'coffeelint',
requirements: 2.1.0,
kind: 'runtime',
platform: 'npm'
}
] |
ah no not additional_dependencies (though maybe that's worth doing too? idk?) -- I was thinking more like if you looked at https://github.com/asottile/t for example it would have dependencies on github.com/pre-commit/pre-commit-hooks, ... based on its consuming I don't know how that gets represented in libraries.io though 🤔 |
oh ok, if i understood the documentation correct, this is done by bibliothecary (if you log in to libraries.io, this will check your repositories for dependencies). i added pre-commit to bibliothecary here. |
neat, that looks good -- though you'll want to skip |
yeah thanks, i'm going to to change this. |
i just tried adding the parsing of
let me know what you think about adding this or if you have any suggestions |
feel free to skip additional_dependencies 👍 |
all-repos.json
and maybe(?) take some inspiration from the go implementation for libraries.io (since the distribution mechanism is similar)The text was updated successfully, but these errors were encountered: