Skip to content

Commit 1808308

Browse files
feat: add PR self-review reminder workflow and fix import path for MultiSelectFilter
1 parent f1d2244 commit 1808308

2 files changed

Lines changed: 40 additions & 2 deletions

File tree

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
name: PR Self-Review Reminder
2+
3+
on:
4+
pull_request:
5+
types: [opened, ready_for_review]
6+
7+
jobs:
8+
remind-self-review:
9+
runs-on: ubuntu-latest
10+
steps:
11+
- name: PR self-review reminder
12+
uses: actions/github-script@v6
13+
with:
14+
script: |
15+
const prNumber = context.payload.pull_request.number;
16+
17+
const body = `
18+
Thank you for opening this PR!
19+
20+
Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools.
21+
22+
Please take a moment to:
23+
- Check the “Files changed” tab
24+
- Leave comments on any lines for functions, comments, etc. that are important, non-obvious, or may need attention
25+
- Clarify decisions you made or areas you might be unsure about and/or any future updates being considered.
26+
- Finally submit all the comments!
27+
28+
This helps make the review process smoother and gives us a clearer understanding of your thought process.
29+
30+
Once you've added your self-review, we'll continue from our side. Thank you!
31+
`;
32+
33+
await github.rest.issues.createComment({
34+
owner: context.repo.owner,
35+
repo: context.repo.repo,
36+
issue_number: prNumber,
37+
body: body
38+
});

frontend/src/components/HomeComponents/Tasks/__tests__/Tasks.test.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -342,7 +342,7 @@ describe('Tasks Component', () => {
342342

343343
expect(await screen.findByText('Mocked BottomBar')).toBeInTheDocument();
344344

345-
const multiSelectFilter = require('@/components/ui/multiSelect');
345+
const multiSelectFilter = require('@/components/ui/multi-select');
346346

347347
expect(multiSelectFilter.MultiSelectFilter).toHaveBeenCalledWith(
348348
expect.objectContaining({
@@ -355,7 +355,7 @@ describe('Tasks Component', () => {
355355

356356
test('filters tasks to show only overdue tasks when status "overdue" is selected', async () => {
357357
const MultiSelectFilter =
358-
require('@/components/ui/multiSelect').MultiSelectFilter;
358+
require('@/components/ui/multi-select').MultiSelectFilter;
359359

360360
MultiSelectFilter.mockImplementation(({ title }: { title: string }) => {
361361
return <div data-testid={`ms-${title}`}>Mocked MultiSelect: {title}</div>;

0 commit comments

Comments
 (0)