Skip to content

adding_flagcheck_for_asisctf2013_dice - #170

Open
x-64 wants to merge 2 commits into
pwncollege:mainfrom
x-64:adding_flagcheck_for_asisctf2013_dice
Open

adding_flagcheck_for_asisctf2013_dice#170
x-64 wants to merge 2 commits into
pwncollege:mainfrom
x-64:adding_flagcheck_for_asisctf2013_dice

Conversation

@x-64

@x-64 x-64 commented Jan 29, 2026

Copy link
Copy Markdown
Contributor
grafik
echo -n ASIS_e8a1...d534 | sha256sum 
37ac869c890747539b9eabba5bae7558ca18d9ae46f79f25cdd6c54707d5f587 

@alchemy1729-bot

alchemy1729-bot commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

Recommended verdict: reject.

Reasoning:

  • I compared the submitted checker against the current challenge directory state.
  • The new checker encodes a different acceptance condition than the one already recorded for this challenge, and the PR does not update the challenge consistently as a unit.
  • That leaves the challenge internally inconsistent, so I do not recommend merging it as submitted.

Decision not final, awaiting maintainer input.

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