Skip to content

Add /pr workflow for AI-assisted pull requests#9

Merged
Laoujin merged 1 commit intomasterfrom
feature/claude-pr-workflow
Feb 25, 2026
Merged

Add /pr workflow for AI-assisted pull requests#9
Laoujin merged 1 commit intomasterfrom
feature/claude-pr-workflow

Conversation

@Laoujin
Copy link
Copy Markdown
Member

@Laoujin Laoujin commented Feb 25, 2026

Summary

  • Update .gitignore to share .claude/commands/ with the team (personal settings still ignored)
  • Add /pr slash command that creates a PR and posts an AI code review directly to GitHub

Test plan

  • Run /pr from a feature branch
  • Verify PR is created with summary
  • Verify AI review is posted as a PR review comment

🤖 Generated with Claude Code

- Update .gitignore to share .claude/commands (keep settings.local.json ignored)
- Add /pr command that creates PR and posts AI code review to GitHub

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Copy Markdown
Member Author

@Laoujin Laoujin left a comment

Choose a reason for hiding this comment

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

AI Code Review 🤖

Files reviewed: .gitignore, .claude/commands/pr.md

Findings

No blocking issues found.

Observations:

  • .gitignore change correctly narrows the ignore pattern from all of .claude/ to just settings.local.json
  • The /pr command workflow is well-structured with clear steps
  • Uses gh pr review for native GitHub integration

Minor suggestions (non-blocking):

  • Consider adding error handling guidance for when gh CLI is not authenticated
  • Could add a step to check if PR already exists for the branch

✅ Code looks good - ready for human review.

@Laoujin Laoujin merged commit 197e586 into master Feb 25, 2026
2 checks passed
Comment thread .claude/commands/pr.md
- Run `git status` and `git log origin/master..HEAD --oneline` to see what will be in the PR
- If on master, stop and ask user to create a feature branch first
- Push branch if not yet pushed: `git push -u origin HEAD`

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Consider adding a step to verify gh auth status before running gh commands.

Laoujin added a commit that referenced this pull request Mar 11, 2026
Add /pr workflow for AI-assisted pull requests
NicolasHub referenced this pull request in itenium-be/Bootcamp-AI-MidnightBlue Mar 13, 2026
Adds a coach-facing team overview with all consultants in one view,
inactive highlighting (>21 days), and quick navigation to each profile.

Backend:
- ConsultantProfileEntity: links a learner user to their team + tracks LastActivityAt
- EF migration AddConsultantProfile
- GET /api/consultant — returns team consultants (coach sees own team; backoffice sees all)
  with IsInactive flag and DaysSinceActivity computed fields
- GET /api/consultant/{userId} — consultant detail; team-scoped for managers
- Seed data: 11 consultant users across 4 teams with varied activity dates
  (5 inactive, 6 active) to demonstrate the dashboard live on demo day
- 8 new TDD integration tests — 22 total, all passing

Frontend:
- TeamMembers page (/team/members): consultants grouped into
  "Needs Attention" (inactive ≥21 days) and "Active" sections,
  each card shows name, email, team badge, activity badge, and "View Profile" link
- ConsultantProfile page (/consultant/:userId): detail view with
  activity timeline, joined date, last active, status
- EN + NL i18n keys; nav label updated to "My Consultants"

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
JoachimDhondtItenium referenced this pull request in itenium-be/Bootcamp-AI-MidnightBlue Mar 13, 2026
MVP : prd.md #1, #2, #4, #9 geimplementeerd
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.

1 participant