Skip to content

[AIReview | js_array-method-sort]: Add .reviewrelatedfiles and checklist.md#2504

Merged
Vasya564 merged 2 commits intomasterfrom
PF-13709-add-reviewrelatedfiles
Apr 17, 2026
Merged

[AIReview | js_array-method-sort]: Add .reviewrelatedfiles and checklist.md#2504
Vasya564 merged 2 commits intomasterfrom
PF-13709-add-reviewrelatedfiles

Conversation

@Vasya564
Copy link
Copy Markdown
Contributor

Summary

AI reviewer hallucinates requirements on PRs here because it doesn't see src/arrayMethodSort.test.js. The test file only covers sort2, but the reviewer assumes students must also override Array.prototype.sort.

  • Add .reviewrelatedfiles with src/**/* — same pattern used by 17 other repos (e.g. js_get_coin_combination, js_split_integer). AIReview service picks up files matching these globs and sends them as extra LLM context.
  • Add checklist.md with task-specific items so the reviewer cites real checklist numbers instead of inventing them.

PF-13709

Copy link
Copy Markdown

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Choose a reason for hiding this comment

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

Sorry, the pull request review was unsuccessful for the following reasons:

Warning

There are no files to review. Please make sure that you have added files to the pull request. Some files and directories may be ignored from the review or failed to load.
[CODE: NFR]

Additional info:
Ignored files list:

  • .reviewrelatedfiles
  • checklist.md

If the issue is within your control, please address it.
The next review attempt will automatically launch in about 1 hour.
If you believe this is an error or need assistance, please contact the support team.

@Vasya564 Vasya564 marked this pull request as ready for review April 17, 2026 11:36
@Vasya564 Vasya564 requested a review from marina-tilniak April 17, 2026 11:36
@Vasya564 Vasya564 merged commit adcb7ff into master Apr 17, 2026
1 check failed
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.

3 participants