Skip to content

Address PR review feedback: fix indentation, update configuration paths, replace archived server#169

Merged
d-morrison merged 2 commits intotest-mcpfrom
copilot/sub-pr-163-again
Jan 8, 2026
Merged

Address PR review feedback: fix indentation, update configuration paths, replace archived server#169
d-morrison merged 2 commits intotest-mcpfrom
copilot/sub-pr-163-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 8, 2026

Applies code review suggestions from PR #163 to fix workflow syntax errors, improve configurability, and update deprecated dependencies.

Workflow Fixes

  • pkgdown.yaml: Fixed indentation in rjags verification block (lines 92-96) - was 11 spaces, now 10 to match yaml structure

Configuration Updates

  • .vscode/settings.json: Replaced hardcoded R-4.1.0 paths with placeholder R-x.y.z and added comments instructing Windows users to update for their R version
  • .github/mcp/mcp-config.json: Added $comment field documenting that hardcoded paths (/home/runner/work/serodynamics/serodynamics) are CI-specific and must be updated for local use
  • .github/mcp/README.md: Added prominent "Important: Update Repository Paths" section with step-by-step instructions

Dependency Updates

  • mcp-config.json: Replaced archived @modelcontextprotocol/server-brave-search with official @brave/brave-search-mcp-server
  • .devcontainer/setup.sh: Removed redundant pak availability check (already installed in line 38)

The GitHub MCP server respects token permissions and branch protection rules - it cannot bypass protected branch restrictions on main.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…al Brave Search server

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix pkgdown website rendering issues Address PR review feedback: fix indentation, update configuration paths, replace archived server Jan 8, 2026
Copilot AI requested a review from d-morrison January 8, 2026 06:04
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 8, 2026

📕 Preview documentation for this PR has been cleaned up.

@d-morrison d-morrison marked this pull request as ready for review January 8, 2026 06:47
@d-morrison d-morrison merged commit 3c7d7a0 into test-mcp Jan 8, 2026
12 checks passed
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