Skip to content

Commit 35e41ff

Browse files
committed
fix: use correct /ixr/ci path for app links
1 parent de0fda2 commit 35e41ff

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

src/main.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ async function runInCI(
6464
process.env.SITE_APP_URL ??
6565
siteApiEndpoint.replace(/\/api\/?$/, "").replace("api.", "app.");
6666
const baseUrl = appUrl.replace(/\/$/, "");
67-
reportContext.runUrl = `${baseUrl}/ci/${runId}`;
67+
reportContext.runUrl = `${baseUrl}/ixr/ci/${runId}`;
6868
reportContext.queryBaseUrl = baseUrl;
6969
}
7070

src/reporters/github/success.md.j2

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,15 +14,15 @@
1414
#### This PR improves queries
1515

1616
{% for q in displayImproved %}
17-
- {% if queryBaseUrl %}[<code>{{ q.queryPreview }}</code>]({{ queryBaseUrl }}/ci/queries/{{ q.hash }}){% else %}<code>{{ q.queryPreview }}</code>{% endif %}<br>cost {{ formatCost(q.previousCost) }} → {{ formatCost(q.currentCost) }} ({{ q.improvementPercentage | round(0) }}% reduction){% if q.indexesChanged %}{% if q.previousIndexes.length > 0 %}<br>was using: {% for idx in q.previousIndexes %}<code>{{ idx }}</code>{% if not loop.last %}, {% endif %}{% endfor %}{% endif %}{% if q.currentIndexes.length > 0 %}<br>now using: {% for idx in q.currentIndexes %}<code>{{ idx }}</code>{% if not loop.last %}, {% endif %}{% endfor %}{% endif %}{% endif %}{{""}}
17+
- {% if queryBaseUrl %}[<code>{{ q.queryPreview }}</code>]({{ queryBaseUrl }}/ixr/ci/queries/{{ q.hash }}){% else %}<code>{{ q.queryPreview }}</code>{% endif %}<br>cost {{ formatCost(q.previousCost) }} → {{ formatCost(q.currentCost) }} ({{ q.improvementPercentage | round(0) }}% reduction){% if q.indexesChanged %}{% if q.previousIndexes.length > 0 %}<br>was using: {% for idx in q.previousIndexes %}<code>{{ idx }}</code>{% if not loop.last %}, {% endif %}{% endfor %}{% endif %}{% if q.currentIndexes.length > 0 %}<br>now using: {% for idx in q.currentIndexes %}<code>{{ idx }}</code>{% if not loop.last %}, {% endif %}{% endfor %}{% endif %}{% endif %}{{""}}
1818
{% endfor %}
1919
{% endif %}
2020

2121
{% if displayRegressed.length > 0 %}
2222
#### This PR has regressions on queries
2323

2424
{% for q in displayRegressed %}
25-
- {% if queryBaseUrl %}[<code>{{ q.queryPreview }}</code>]({{ queryBaseUrl }}/ci/queries/{{ q.hash }}){% else %}<code>{{ q.queryPreview }}</code>{% endif %}<br>cost {{ formatCost(q.previousCost) }} → {{ formatCost(q.currentCost) }} (+{{ q.regressionPercentage | round(0) }}%)
25+
- {% if queryBaseUrl %}[<code>{{ q.queryPreview }}</code>]({{ queryBaseUrl }}/ixr/ci/queries/{{ q.hash }}){% else %}<code>{{ q.queryPreview }}</code>{% endif %}<br>cost {{ formatCost(q.previousCost) }} → {{ formatCost(q.currentCost) }} (+{{ q.regressionPercentage | round(0) }}%)
2626
{% endfor %}
2727
{% endif %}
2828

@@ -40,7 +40,7 @@
4040
#### This PR introduces queries with recommendations
4141

4242
{% for r in displayRecommendations %}
43-
- {% if queryBaseUrl %}[<code>{{ r.queryPreview }}</code>]({{ queryBaseUrl }}/ci/queries/{{ r.fingerprint }}){% else %}<code>{{ r.queryPreview }}</code>{% endif %}<br>recommended index <code>{{ r.proposedIndexes | join("</code>, <code>") }}</code><br>cost {{ formatCost(r.baseCost) }} → {{ formatCost(r.optimizedCost) }} ({{ (((r.baseCost - r.optimizedCost) / r.baseCost) * 100) | round(0) }}% reduction)
43+
- {% if queryBaseUrl %}[<code>{{ r.queryPreview }}</code>]({{ queryBaseUrl }}/ixr/ci/queries/{{ r.fingerprint }}){% else %}<code>{{ r.queryPreview }}</code>{% endif %}<br>recommended index <code>{{ r.proposedIndexes | join("</code>, <code>") }}</code><br>cost {{ formatCost(r.baseCost) }} → {{ formatCost(r.optimizedCost) }} ({{ (((r.baseCost - r.optimizedCost) / r.baseCost) * 100) | round(0) }}% reduction)
4444
{% endfor %}
4545
{% endif %}
4646

@@ -49,7 +49,7 @@
4949
<summary>{{ preExistingRecommendations.length }} pre-existing issue{{ "s" if preExistingRecommendations.length != 1 else "" }}</summary>
5050

5151
{% for r in preExistingRecommendations %}
52-
- {% if queryBaseUrl %}[<code>{{ r.queryPreview }}</code>]({{ queryBaseUrl }}/ci/queries/{{ r.fingerprint }}){% else %}<code>{{ r.queryPreview }}</code>{% endif %}<br>index <code>{{ r.proposedIndexes | join("</code>, <code>") }}</code><br>cost {{ formatCost(r.baseCost) }} → {{ formatCost(r.optimizedCost) }} ({{ (((r.baseCost - r.optimizedCost) / r.baseCost) * 100) | round(0) }}% reduction)
52+
- {% if queryBaseUrl %}[<code>{{ r.queryPreview }}</code>]({{ queryBaseUrl }}/ixr/ci/queries/{{ r.fingerprint }}){% else %}<code>{{ r.queryPreview }}</code>{% endif %}<br>index <code>{{ r.proposedIndexes | join("</code>, <code>") }}</code><br>cost {{ formatCost(r.baseCost) }} → {{ formatCost(r.optimizedCost) }} ({{ (((r.baseCost - r.optimizedCost) / r.baseCost) * 100) | round(0) }}% reduction)
5353
{% endfor %}
5454
</details>
5555
{% endif %}

0 commit comments

Comments
 (0)