Skip to content

Added CWPE#171

Closed
janssensjelle wants to merge 0 commit intohackthebox:mainfrom
janssensjelle:main
Closed

Added CWPE#171
janssensjelle wants to merge 0 commit intohackthebox:mainfrom
janssensjelle:main

Conversation

@janssensjelle
Copy link
Contributor

@janssensjelle janssensjelle commented Feb 9, 2026

Types of changes

What types of changes does your code introduce?
Put an x in the boxes that apply.

  • Bugfix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality not to work as expected).
  • Documentation Update (if none of the other choices applies).

Proposed changes

Added CWPE cert.

Please add the CPWE discord role id: 1466407288758599821

Checklist

Put an x in the boxes that apply.

  • I have read and followed the CONTRIBUTING.md
    doc.
  • Lint and unit tests pass locally with my changes.
  • I have added the necessary documentation (if appropriate).

Additional context

@codecov
Copy link

codecov bot commented Feb 12, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.37%. Comparing base (131c32b) to head (9c13bf2).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
src/helpers/verification.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #171      +/-   ##
==========================================
- Coverage   59.38%   59.37%   -0.02%     
==========================================
  Files          50       50              
  Lines        2903     2907       +4     
==========================================
+ Hits         1724     1726       +2     
- Misses       1179     1181       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dimoschi
Copy link
Contributor

dimoschi commented Mar 4, 2026

@janssensjelle the PR is empty, I guess that's not intentional?

@janssensjelle
Copy link
Contributor Author

@dimoschi this PR was most definitely not empty when I made it. Something weird happened, watch here: c0f26a9

@janssensjelle
Copy link
Contributor Author

@dimoschi applied the changes again

@dimoschi
Copy link
Contributor

dimoschi commented Mar 24, 2026

@dimoschi applied the changes again

Sorry this took longer than it should!

Copy link
Contributor

@dimoschi dimoschi left a comment

Choose a reason for hiding this comment

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

LGTM

@dimoschi
Copy link
Contributor

@janssensjelle indeed something weird. I've closed because there are no commits, but can you please re-open it properly and I promise I will merge it ASAP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants