Skip to content
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

SDK: fix dependencies #132

Merged
merged 1 commit into from
Sep 10, 2024
Merged

SDK: fix dependencies #132

merged 1 commit into from
Sep 10, 2024

Conversation

squioc
Copy link
Collaborator

@squioc squioc commented Sep 10, 2024

Move jsonschema dependency in the main dependencies group as it's required for the CLI.
Currently, the CLI doesn't work in the version 1.14.0 of the sdk

$ sekoia-automation synchronize-library --check-image-on-registry --module "Ubika"
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.11.9/x64/bin/sekoia-automation", line 5, in <module>
    from sekoia_automation.cli import app
  File "/opt/hostedtoolcache/Python/3.11.9/x64/lib/python3.11/site-packages/sekoia_automation/cli.py", line 8, in <module>
    from sekoia_automation.scripts.action_runner import ModuleItemRunner
  File "/opt/hostedtoolcache/Python/3.11.9/x64/lib/python3.11/site-packages/sekoia_automation/scripts/action_runner.py", line 9, in <module>
    from jsonschema import validate
ModuleNotFoundError: No module named 'jsonschema'

Move jsonschema dependency in the main dependencies group as it's required for the CLI
@squioc squioc added the bug Something isn't working label Sep 10, 2024
Copy link

Test Results

176 tests  ±0   175 ✅ +1   1m 31s ⏱️ +26s
  1 suites ±0     1 💤 ±0 
  1 files   ±0     0 ❌  - 1 

Results for commit 4537405. ± Comparison against base commit 8253e22.

@Darkheir Darkheir merged commit 78be8e6 into main Sep 10, 2024
4 checks passed
@otetard otetard deleted the chore/FixDependency branch September 29, 2024 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants