You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The following four placeholders are present in the stubs. They should be detected and a "must-fix" message should be added to remove the line which is now dead-code.
thrownewError('Please implement the (...) function')thrownewError('Remove this line and implement the function')thrownewError('Implement the (...) function')thrownewError('Remove this statement and implement this function');
I was investigating this issue and noticed that isStubThrowStatement in src/analyzers/utils/extract_main_method.ts already checks for all 4 placeholder strings mentioned in the issue:
**- 'Please implement'
'Remove this line and implement'
'Implement the'
'Remove this statement and implement'**
The function triggers the REMOVE_STUB_THROW essential comment when these are detected. It looks like this functionality might already be implemented. Should this issue be closed?
Activity
IsaacG commentedon Jun 24, 2025
SleeplessByte commentedon Jun 24, 2025
jagdish-15 commentedon Jun 25, 2025
SleeplessByte commentedon Jul 4, 2025
Cool-Katt commentedon Jul 4, 2025
SleeplessByte commentedon Jul 4, 2025
jagdish-15 commentedon Jul 4, 2025
jagdish-15 commentedon Jul 5, 2025
naderakhgari commentedon May 6, 2026
Hi @SleeplessByte, I can fix this issue in PR but I think I don't have access to raise a PR. Is this still a valid issue to fix?
SleeplessByte commentedon May 6, 2026
Yes, ping me in the pr and i can reopen it
IsaacG commentedon May 6, 2026
You can open a PR. The bot will auto-close it. If it's been previously approved, we can reopen it.
bpradhan-dev commentedon Sep 5, 2026
Hi @SleeplessByte!
I was investigating this issue and noticed that
isStubThrowStatementinsrc/analyzers/utils/extract_main_method.tsalready checks for all 4 placeholder strings mentioned in the issue:**- 'Please implement'
The function triggers the
REMOVE_STUB_THROWessential comment when these are detected. It looks like this functionality might already be implemented. Should this issue be closed?Thanks!