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

fix(kernel_crawler): enforce kernelversion to be a string. #167

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

FedeDP
Copy link
Collaborator

@FedeDP FedeDP commented Aug 3, 2023

What type of PR is this?

/kind bug

Any specific area of the project related to this PR?

/area crawler

What this PR does / why we need it:

Part 2 of #165 :D
I forgot a couple of occurrences of int kernelversion.
Moreover, i also added a type check when building the repo.DriverkitConfig object.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Signed-off-by: Federico Di Pierro <nierro92@gmail.com>
@FedeDP
Copy link
Collaborator Author

FedeDP commented Aug 3, 2023

/cc @EXONER4TED

@poiana poiana requested a review from EXONER4TED August 3, 2023 15:04
@FedeDP
Copy link
Collaborator Author

FedeDP commented Aug 3, 2023

/cc @maxgio92 :P

Copy link
Member

@maxgio92 maxgio92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@FedeDP
Copy link
Collaborator Author

FedeDP commented Aug 3, 2023

The error is unrelated from this PR:

File "/home/runner/.local/lib/python3.10/site-packages/kernel_crawler/utils/download.py", line 26, in get_url
return zlib.decompress(resp.content, 47)
zlib.error: Error -3 while decompressing data: incorrect header check

and it's working just fine locally.

@poiana poiana merged commit bb69a6f into main Aug 3, 2023
2 checks passed
@poiana poiana deleted the chore/enforce_str_for_kernelversion branch August 3, 2023 18:12
@FedeDP FedeDP restored the chore/enforce_str_for_kernelversion branch October 23, 2023 13:42
@FedeDP FedeDP deleted the chore/enforce_str_for_kernelversion branch October 23, 2023 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants