GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — test(fastmcp): Use `TestClient` for Streamable HTTP by alexander-alderman-webb · Pull Request #5384 · getsentry/sentry-python

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Jan 28, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator

Description

Stop mocking transport layer of mcp package in tests.

Issues

Reminders

  • Please add tests to validate your changes, and lint your code using tox -e linters.
  • Add GH Issue ID & Linear ID (if applicable)
  • PR title should use conventional commit style (feat:, fix:, ref:, meta:)
  • For external contributors: CONTRIBUTING.md, Sentry SDK development docs, Discord community

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

sentry[bot] reacted with hooray emoji
All reactions
  • 🎉 1 reaction
alexander-alderman-webb added 11 commits January 16, 2026 15:19
@alexander-alderman-webb
test(mcp): Simulate stdio transport with memory streams
a8372b0
@alexander-alderman-webb
test(fastmcp): Simulate stdio transport with memory streams
87a2e26
@alexander-alderman-webb
cleaning up
70edb6d
@alexander-alderman-webb
Merge branch 'webb/test-mcp-stdio' into webb/test-fastmcp-stdio
c238f84
@alexander-alderman-webb
send initialization response
f1362bd
@alexander-alderman-webb
Merge branch 'webb/test-mcp-stdio' into webb/test-fastmcp-stdio
34dba91
@alexander-alderman-webb
cleanup
46659c5
@alexander-alderman-webb
gate import in conftest
ac8e6e4
@alexander-alderman-webb
test(mcp): Use TestClient for Streamable HTTP
2766b40
@alexander-alderman-webb
test(fastmcp): Use TestClient for Streamable HTTP
d3afb38
@alexander-alderman-webb
clean up merge
270636d
@github-actions

github-actions Bot commented Jan 28, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

Span Streaming

  • feat(span-streaming): Add spans to telemetry pipeline, add span name and attributes (3) by sentrivana in #5399
  • feat(span-streaming): Add span batcher (2) by sentrivana in #5398
  • feat(span-streaming): Add experimental trace_lifecycle switch (1) by sentrivana in #5397

Bug Fixes 🐛

  • fix(arq): handle settings_cls passed as keyword argument by nc9 in #5393
  • fix(dramatiq): cleanup isolated scope and transaction when message is skipped by frankie567 in #5346
  • fix(google-genai): deactivate google genai when langchain is used by shellmayr in #5389

Internal Changes 🔧

Fastmcp

  • test(fastmcp): Use TestClient for Streamable HTTP by alexander-alderman-webb in #5384
  • test(fastmcp): Simulate stdio transport with memory streams by alexander-alderman-webb in #5333

Mcp

  • test(mcp): Use TestClient for Streamable HTTP by alexander-alderman-webb in #5383
  • test(mcp): Remove unused stdio helpers by alexander-alderman-webb in #5409
  • test(mcp): Simulate stdio transport with memory streams by alexander-alderman-webb in #5329

Other

  • ci: migration to the new codecov action by MathurAditya724 in #5392

🤖 This preview updates automatically when you update the PR.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

alexander-alderman-webb added 8 commits January 30, 2026 13:33
@alexander-alderman-webb
remove mixed method test
8c9aa86
@alexander-alderman-webb
rename tests
6579935
@alexander-alderman-webb
merge
8aa6363
@alexander-alderman-webb
rename tests
30e9ec3
@alexander-alderman-webb
Merge branch 'webb/test-fastmcp-stdio' into webb/test-mcp-streamable-…
aaf69c3
…http
@alexander-alderman-webb
merge
0ab862c
@alexander-alderman-webb
remove mocks
1380f56
@alexander-alderman-webb
Merge branch 'webb/test-mcp-streamable-http' into webb/test-fastmcp-s…
0e37050
…treamable-http
@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review January 30, 2026 13:07
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner January 30, 2026 13:07
cursor[bot]
cursor Bot reviewed Jan 30, 2026
View reviewed changes
Comment thread tests/integrations/fastmcp/test_fastmcp.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread tests/integrations/fastmcp/test_fastmcp.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread tests/integrations/fastmcp/test_fastmcp.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread tests/integrations/fastmcp/test_fastmcp.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

alexander-alderman-webb added 5 commits January 30, 2026 14:25
@alexander-alderman-webb
remove test
5976135
@alexander-alderman-webb
Merge branch 'master' into webb/test-mcp-stdio
317c002
@alexander-alderman-webb
merge and remove duplicate asyncio marker
9097b69
@alexander-alderman-webb
Merge branch 'webb/test-fastmcp-stdio' into webb/test-mcp-streamable-…
7703379
…http
@alexander-alderman-webb
merge and remove duplicate import
1133d9e
@github-actions

github-actions Bot commented Jan 30, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ 13 passed | Total: 13 | Pass Rate: 100% | Execution Time: 8.79s

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 13716 uncovered lines.

Files with missing lines (180)
File Patch % Lines
langchain.py 3.28% ⚠️ 590 Missing
utils.py 52.52% ⚠️ 434 Missing and 79 partials
openai.py 5.63% ⚠️ 486 Missing
utils.py 0.00% ⚠️ 479 Missing
__init__.py 5.08% ⚠️ 374 Missing
tracing_utils.py 45.27% ⚠️ 324 Missing and 28 partials
starlette.py 5.34% ⚠️ 337 Missing
scope.py 70.52% ⚠️ 242 Missing and 66 partials
transport.py 23.54% ⚠️ 302 Missing and 2 partials
client.py 55.98% ⚠️ 221 Missing and 56 partials
mcp.py 4.85% ⚠️ 255 Missing
anthropic.py 9.43% ⚠️ 240 Missing
transaction_profiler.py 35.52% ⚠️ 216 Missing and 10 partials
utils.py 16.34% ⚠️ 215 Missing
langgraph.py 5.29% ⚠️ 215 Missing
span_processor.py 0.00% ⚠️ 205 Missing
__init__.py 5.14% ⚠️ 203 Missing
continuous_profiler.py 43.45% ⚠️ 177 Missing and 17 partials
strawberry.py 8.54% ⚠️ 182 Missing
tracing.py 71.37% ⚠️ 136 Missing and 38 partials
aws_lambda.py 16.50% ⚠️ 167 Missing
huggingface_hub.py 8.89% ⚠️ 164 Missing
sanic.py 9.60% ⚠️ 160 Missing
aiohttp.py 10.84% ⚠️ 148 Missing
cloud_resource_context.py 0.00% ⚠️ 145 Missing
ai_client.py 0.00% ⚠️ 145 Missing
rust_tracing.py 0.00% ⚠️ 143 Missing
litellm.py 0.00% ⚠️ 138 Missing
litestar.py 9.59% ⚠️ 132 Missing
starlite.py 8.33% ⚠️ 132 Missing
falcon.py 8.94% ⚠️ 112 Missing
flask.py 12.50% ⚠️ 112 Missing
spotlight.py 28.47% ⚠️ 103 Missing and 8 partials
asgi.py 20.71% ⚠️ 111 Missing
cohere.py 12.70% ⚠️ 110 Missing
agent_run.py 0.00% ⚠️ 110 Missing
arq.py 9.92% ⚠️ 109 Missing
asgi.py 0.00% ⚠️ 109 Missing
hub.py 45.60% ⚠️ 105 Missing and 3 partials
envelope.py 54.59% ⚠️ 89 Missing and 17 partials
pymongo.py 10.17% ⚠️ 106 Missing
caching.py 0.00% ⚠️ 106 Missing
utils.py 0.00% ⚠️ 103 Missing
templates.py 0.00% ⚠️ 100 Missing
asyncpg.py 11.61% ⚠️ 99 Missing
quart.py 16.10% ⚠️ 99 Missing
wsgi.py 22.66% ⚠️ 99 Missing
gcp.py 0.00% ⚠️ 98 Missing
otlp.py 0.00% ⚠️ 97 Missing
utils.py 13.39% ⚠️ 97 Missing
sessions.py 27.82% ⚠️ 96 Missing
pyramid.py 13.76% ⚠️ 94 Missing
tornado.py 14.55% ⚠️ 94 Missing
__init__.py 0.00% ⚠️ 93 Missing
models.py 5.15% ⚠️ 92 Missing
bottle.py 11.65% ⚠️ 91 Missing
middleware.py 0.00% ⚠️ 90 Missing
agent_run.py 0.00% ⚠️ 90 Missing
_wsgi_common.py 30.71% ⚠️ 88 Missing and 1 partials
loguru.py 11.58% ⚠️ 84 Missing
asyncio.py 0.00% ⚠️ 80 Missing
invoke_agent.py 0.00% ⚠️ 79 Missing
runner.py 0.00% ⚠️ 77 Missing
session.py 15.56% ⚠️ 76 Missing
clickhouse_driver.py 17.58% ⚠️ 75 Missing
pure_eval.py 0.00% ⚠️ 73 Missing
worker.py 22.58% ⚠️ 72 Missing
ariadne.py 14.46% ⚠️ 71 Missing
gql.py 10.13% ⚠️ 71 Missing
sqlalchemy.py 10.26% ⚠️ 70 Missing
fastapi.py 15.85% ⚠️ 69 Missing
monitoring.py 17.07% ⚠️ 68 Missing
graphene.py 12.82% ⚠️ 68 Missing
transactions.py 0.00% ⚠️ 67 Missing
_queue.py 26.67% ⚠️ 66 Missing
logging.py 66.67% ⚠️ 51 Missing and 15 partials
streaming.py 0.00% ⚠️ 66 Missing
stdlib.py 60.45% ⚠️ 53 Missing and 11 partials
boto3.py 14.86% ⚠️ 63 Missing
httpx.py 16.44% ⚠️ 61 Missing
chalice.py 16.18% ⚠️ 57 Missing
propagator.py 0.00% ⚠️ 57 Missing
server.py 0.00% ⚠️ 56 Missing
spark_driver.py 67.47% ⚠️ 54 Missing and 2 partials
api.py 64.43% ⚠️ 53 Missing
graph_nodes.py 0.00% ⚠️ 52 Missing
gnu_backtrace.py 0.00% ⚠️ 51 Missing
socket.py 0.00% ⚠️ 50 Missing
views.py 0.00% ⚠️ 50 Missing
tools.py 0.00% ⚠️ 49 Missing
_batcher.py 41.25% ⚠️ 47 Missing
_asgi_common.py 16.67% ⚠️ 45 Missing
signals_handlers.py 0.00% ⚠️ 44 Missing
invoke_agent.py 0.00% ⚠️ 43 Missing
utils.py 22.22% ⚠️ 42 Missing and 1 partials
threading.py 63.16% ⚠️ 35 Missing and 5 partials
client.py 0.00% ⚠️ 40 Missing
serializer.py 84.95% ⚠️ 28 Missing and 10 partials
executing.py 0.00% ⚠️ 38 Missing
client.py 0.00% ⚠️ 38 Missing
_span_batcher.py 35.71% ⚠️ 36 Missing
serverless.py 0.00% ⚠️ 36 Missing
server.py 0.00% ⚠️ 35 Missing
caches.py 47.46% ⚠️ 31 Missing and 3 partials
sys_exit.py 0.00% ⚠️ 32 Missing
_async_common.py 37.25% ⚠️ 32 Missing
launchdarkly.py 0.00% ⚠️ 31 Missing
trytond.py 0.00% ⚠️ 30 Missing
tools.py 0.00% ⚠️ 30 Missing
integration.py 0.00% ⚠️ 30 Missing
error_tracing.py 0.00% ⚠️ 29 Missing
utils.py 73.33% ⚠️ 20 Missing and 8 partials
scrubber.py 76.81% ⚠️ 16 Missing and 9 partials
ray.py 70.24% ⚠️ 25 Missing
_werkzeug.py 11.11% ⚠️ 24 Missing
typer.py 0.00% ⚠️ 24 Missing
__init__.py 93.89% ⚠️ 14 Missing and 10 partials
tasks.py 0.00% ⚠️ 24 Missing
_compat.py 41.03% ⚠️ 23 Missing
statsig.py 0.00% ⚠️ 23 Missing
execute_tool.py 0.00% ⚠️ 23 Missing
logger.py 43.59% ⚠️ 22 Missing
decorator.py 37.14% ⚠️ 22 Missing
__init__.py 84.78% ⚠️ 14 Missing and 8 partials
ai_client.py 15.38% ⚠️ 22 Missing
_log_batcher.py 0.00% ⚠️ 21 Missing
attachments.py 27.59% ⚠️ 21 Missing
unraisablehook.py 0.00% ⚠️ 21 Missing
__init__.py 8.70% ⚠️ 21 Missing
openfeature.py 0.00% ⚠️ 20 Missing
execute_tool.py 0.00% ⚠️ 20 Missing
dramatiq.py 90.91% ⚠️ 10 Missing and 9 partials
unleash.py 0.00% ⚠️ 19 Missing
_lru_cache.py 43.33% ⚠️ 17 Missing and 1 partials
_openai_completions_api.py 21.74% ⚠️ 18 Missing
model_request.py 0.00% ⚠️ 18 Missing
monitor.py 81.97% ⚠️ 11 Missing and 5 partials
beat.py 92.24% ⚠️ 9 Missing and 7 partials
utils.py 0.00% ⚠️ 16 Missing
redis_cluster.py 52.94% ⚠️ 16 Missing
atexit.py 56.25% ⚠️ 14 Missing and 1 partials
excepthook.py 56.25% ⚠️ 14 Missing and 1 partials
feature_flags.py 56.25% ⚠️ 14 Missing
traces.py 46.15% ⚠️ 14 Missing
__init__.py 27.78% ⚠️ 13 Missing
types.py 0.00% ⚠️ 12 Missing
_types.py 66.67% ⚠️ 10 Missing
_sync_common.py 88.00% ⚠️ 6 Missing and 4 partials
spark_worker.py 89.29% ⚠️ 6 Missing and 4 partials
metrics.py 47.06% ⚠️ 9 Missing
beam.py 94.05% ⚠️ 5 Missing and 3 partials
dedupe.py 85.00% ⚠️ 6 Missing and 2 partials
rq.py 97.44% ⚠️ 2 Missing and 6 partials
_metrics_batcher.py 65.00% ⚠️ 7 Missing
handoff.py 0.00% ⚠️ 7 Missing
consts.py 0.00% ⚠️ 7 Missing
redis_py_cluster_legacy.py 53.33% ⚠️ 7 Missing
queries.py 90.32% ⚠️ 3 Missing and 4 partials
_init_implementation.py 75.00% ⚠️ 6 Missing
_openai_responses_api.py 33.33% ⚠️ 6 Missing
debug.py 91.30% ⚠️ 2 Missing and 2 partials
consts.py 0.00% ⚠️ 4 Missing
__init__.py 20.00% ⚠️ 4 Missing
__init__.py 0.00% ⚠️ 4 Missing
__init__.py 89.66% ⚠️ 3 Missing and 1 partials
__init__.py 0.00% ⚠️ 3 Missing
__init__.py 40.00% ⚠️ 3 Missing
__init__.py 0.00% ⚠️ 3 Missing
__init__.py 0.00% ⚠️ 3 Missing
rb.py 70.00% ⚠️ 3 Missing
redis.py 85.00% ⚠️ 3 Missing
consts.py 99.43% ⚠️ 2 Missing
argv.py 100.00% ⚠️ 2 partials
huey.py 100.00% ⚠️ 2 partials
modules.py 94.12% ⚠️ 1 Missing and 1 partials
utils.py 88.24% ⚠️ 2 Missing
agent_workflow.py 71.43% ⚠️ 2 Missing
api.py 100.00% ⚠️ 1 partials
consts.py 0.00% ⚠️ 1 Missing
consts.py 0.00% ⚠️ 1 Missing

Generated by Codecov Action

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

sentry[bot]
sentry Bot reviewed Jan 30, 2026
View reviewed changes
Comment thread tests/integrations/fastmcp/test_fastmcp.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread tests/integrations/fastmcp/test_fastmcp.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

cursor[bot]
cursor Bot reviewed Jan 30, 2026
View reviewed changes
Comment thread tests/conftest.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread tests/integrations/fastmcp/test_fastmcp.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

alexander-alderman-webb added 9 commits January 30, 2026 14:53
@alexander-alderman-webb
address comments
0bd14e3
@alexander-alderman-webb
add missing import fallback
ef33e1f
@alexander-alderman-webb
Merge branch 'webb/test-fastmcp-stdio' into webb/test-mcp-streamable-…
4715afa
…http
@alexander-alderman-webb
Merge branch 'webb/test-mcp-streamable-http' into webb/test-fastmcp-s…
2df9629
…treamable-http
@alexander-alderman-webb
simplify assertions and use stdio in more tests cases
1bf6876
@alexander-alderman-webb
Merge branch 'webb/test-fastmcp-stdio' into webb/test-mcp-streamable-…
fd2cc42
…http
@alexander-alderman-webb
forgot conftest
771f60e
@alexander-alderman-webb
Merge branch 'webb/test-fastmcp-stdio' into webb/test-mcp-streamable-…
8f57fcd
…http
@alexander-alderman-webb
merge and simplify assertions
cc4a19d
sentry[bot]
sentry Bot reviewed Feb 2, 2026
View reviewed changes
Comment thread tests/conftest.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

cursor[bot]
cursor Bot reviewed Feb 2, 2026
View reviewed changes
Comment thread tests/conftest.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

alexander-alderman-webb added 3 commits February 2, 2026 08:48
@alexander-alderman-webb
fix no-return assertion
ef06a2d
@alexander-alderman-webb
Merge branch 'webb/test-fastmcp-stdio' into webb/test-mcp-streamable-…
955d525
…http
@alexander-alderman-webb
Merge branch 'webb/test-mcp-streamable-http' into webb/test-fastmcp-s…
6b81f86
…treamable-http
sentrivana
sentrivana approved these changes Feb 2, 2026
View reviewed changes
Base automatically changed from webb/test-mcp-streamable-http to master February 2, 2026 10:34
alexander-alderman-webb added 4 commits February 2, 2026 11:35
@alexander-alderman-webb
merge master
5a26c3b
@alexander-alderman-webb
import order
aaf1a55
@alexander-alderman-webb
import order
cdab483
@alexander-alderman-webb
remove unused helpers
16f9b60
cursor[bot]
cursor Bot reviewed Feb 2, 2026
View reviewed changes

@cursor cursor Bot left a comment

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Comment thread tests/integrations/fastmcp/test_fastmcp.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

@alexander-alderman-webb
alexander-alderman-webb merged commit 2a4a289 into master Feb 2, 2026
154 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@alexander-alderman-webb
alexander-alderman-webb deleted the webb/test-fastmcp-streamable-http branch February 2, 2026 10:58
@alexander-alderman-webb alexander-alderman-webb mentioned this pull request Feb 4, 2026
test(fastmcp): Ensure MCP spans are children of HTTP transactions #5296
Closed
This was referenced Feb 4, 2026
publish: getsentry/[email protected] getsentry/publish#7065
Closed
publish: getsentry/[email protected] getsentry/publish#7090
Closed
publish: getsentry/[email protected] getsentry/publish#7091
Closed
@sentry-junior sentry-junior Bot mentioned this pull request Oct 29, 2026
test(mcp): Remove unused select_mcp_transactions fixture #8150
Merged
sentrivana added a commit that referenced this pull request Oct 29, 2026
@sentry-junior @sentrivana
test(mcp): Remove unused select_mcp_transactions fixture (#8150)
cc257a4
### Description

Removes the `select_mcp_transactions` fixture from `tests/conftest.py`.
Nothing has used it since #5292.

#5383 added it as a helper in the MCP tests, and #5384 turned it into a
shared `conftest.py` fixture for the MCP and FastMCP Streamable HTTP
tests. A few days later, #5292 ("Nest MCP spans under HTTP
transactions") switched those tests to
`select_transactions_with_mcp_spans` and removed every use of
`select_mcp_transactions`, but left the definition behind.

**Evidence**
- `git grep select_mcp_transactions` matches only the definition. No
test requests it as an argument, and no `request.getfixturevalue(...)`
call uses that name.
- No open PR or issue mentions it.

This only touches test code, so SDK behavior doesn't change.

#### Issues
n/a

<!-- junior-request-attribution:start -->
via **Ivana Kellyer**.
<!-- junior-request-attribution:end -->

<!-- junior-session-footer:start -->
<!-- junior-conversation-id:slack%3AC0BTREZTS2Z%3A1793270559.842119 -->

--

[View Junior
Session](https://junior-prod.sentry.dev/conversations/slack%3AC0BTREZTS2Z%3A1793270559.842119)
[[Sentry]](https://sentry.sentry.io/explore/conversations/slack%3AC0BTREZTS2Z%3A1793270559.842119/?project=4510944073809921)

<!-- junior-session-footer:end -->

Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>
Co-authored-by: Ivana Kellyer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sentry sentry[bot] sentry[bot] left review comments
@cursor cursor[bot] cursor[bot] left review comments
@sentrivana sentrivana sentrivana approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@alexander-alderman-webb @sentrivana
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.