-
Notifications
You must be signed in to change notification settings - Fork 99
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
chore : Add reuse compliance #2455
Conversation
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.
@hamza221 Thank you very much :)
Please fix the remaining issues regarding prettier and I'm unsure where the failing tests are coming from...
a7efe85
to
d802194
Compare
I'm not sure what's wrong with the unit tests , reuse Job is not reading the reuse.toml for some reason I'll try investigating |
blocked by #2466 |
Signed-off-by: Hamza Mahjoubi <[email protected]>
b270aff
to
e594064
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2455 +/- ##
=========================================
Coverage 44.27% 44.27%
Complexity 860 860
=========================================
Files 73 73
Lines 3268 3268
=========================================
Hits 1447 1447
Misses 1821 1821 |
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.
Great, looks good now :) Thank you very much!
fix #2357