-
Notifications
You must be signed in to change notification settings - Fork 8
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
fix: not imported Source / Element classes are not registered #410
base: main
Are you sure you want to change the base?
Conversation
2c73bc7
to
64547f0
Compare
Code Coverage Summary
Diff against main
Results for commit: c33a8b4 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Trivy scanning results. Report Summary ┌─────────┬──────┬─────────────────┬─────────┐
For OSS Maintainers: VEX NoticeIf you're an OSS maintainer and Trivy has detected vulnerabilities in your project that you believe are not actually exploitable, consider issuing a VEX (Vulnerability Exploitability eXchange) statement. To disable this notice, set the TRIVY_DISABLE_VEX_NOTICE environment variable. uv.lock (uv)Total: 19 (MEDIUM: 10, HIGH: 8, CRITICAL: 1) ┌──────────────────┬────────────────┬──────────┬────────┬───────────────────┬───────────────┬──────────────────────────────────────────────────────────────┐ |
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.
how does it solve the issue?
you can specify imports in your pyproject.toml |
can you give me example? |
3fdd057
to
c33a8b4
Compare
|
||
```toml | ||
[tool.ragbits.core] | ||
modules_to_import = {sources=["python.path.to.custom_source:CustomSource"]} |
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.
It's not clear to me why this requires individual objects to be specified (when the module is important all the objects in it are initialised, so the :CustomSource
part here doesn't do anything).
I'm also surprised (but on the fence) about the sources
part. I imagined this as simply:
modules_to_import = [
"python.path.to.custom_source",
]
No description provided.