Change Contact Support ticket form from Zendesk to Pylon#200
Change Contact Support ticket form from Zendesk to Pylon#200DMarinhoCodacy wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the gitignore file to ignore shell scripts, updates several tool versions in the Codacy configuration (including replacing semgrep with opengrep), and updates the 'Contact support' link in the SupportTree view to point to a new Pylon form. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Duplication | 0 |
🟢 Coverage 0.00% diff coverage · +0.00% coverage variation
Metric Results Coverage variation ✅ +0.00% coverage variation Diff coverage ✅ 0.00% diff coverage Coverage variation details
Coverable lines Covered lines Coverage Common ancestor commit (366ba1e) 6941 1733 24.97% Head commit (a37ffb8) 6941 (+0) 1733 (+0) 24.97% (+0.00%) Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch:
<coverage of head commit> - <coverage of common ancestor commit>Diff coverage details
Coverable lines Covered lines Diff coverage Pull request (#200) 1 0 0.00% Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified:
<covered lines added or modified>/<coverable lines added or modified> * 100%
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
Pull Request Overview
This PR updates the 'Contact support' link to the Pylon form. While the change aligns with the primary goal, the PR lacks any description and includes undocumented changes to CI tool versions and the switch from Semgrep to Opengrep. Although Codacy identifies the PR as being up to standards, the functional change in SupportTree.ts is completely uncovered by tests, which constitutes an implementation gap for the required verification scenarios.
About this PR
- The PR description is empty. Please provide context regarding the changes and the motivation behind the CI tooling updates.
Test suggestions
- Verify that SupportTree.getChildren returns a SupportTreeNode for 'Contact support' with the correct Pylon URL
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify that SupportTree.getChildren returns a SupportTreeNode for 'Contact support' with the correct Pylon URL
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
| 'https://docs.codacy.com/codacy-guardrails/codacy-guardrails-getting-started/' | ||
| ), | ||
| new SupportTreeNode('Contact support', 'mail', 'https://codacy.zendesk.com/hc/en-us/requests/new'), | ||
| new SupportTreeNode('Contact support', 'mail', 'https://portal.usepylon.com/codacy/forms/vscode-plugin-form'), |
There was a problem hiding this comment.
⚪ LOW RISK
Suggestion: This change replaces the Zendesk support URL with a Pylon URL. The modified line is currently uncovered by tests. Adding a test case would help verify that the support tree correctly exposes the new Pylon form.\n\nTry running the following prompt in your IDE agent:\n> Create a unit test for the getChildren method in SupportTree.ts to verify it returns a node with the label 'Contact support' and the URL 'https://portal.usepylon.com/codacy/forms/vscode-plugin-form'.
There was a problem hiding this comment.
Pull Request Overview
This pull request updates the 'Contact support' link in the VS Code extension from Zendesk to a Pylon-based form. While the change correctly targets the specified URL, the PR lacks a description providing context for the migration. Furthermore, although the code is reported as being up to standards by Codacy, the modified file src/views/SupportTree.ts has 0% test coverage. This lack of testing prevents automated verification of the URL update and represents a risk for future regressions in the support tree logic.
About this PR
- The modified file
src/views/SupportTree.tshas 0% code coverage. It is recommended to add a unit test to verify that the 'Contact support' node correctly returns the Pylon URL. - The PR is missing a description. Please provide context regarding the migration from Zendesk to Pylon to assist with documentation and future maintenance.
Test suggestions
- Verify that the SupportTree component returns the correct Pylon URL for the 'Contact support' node
- Address 0% coverage in src/views/SupportTree.ts
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify that the SupportTree component returns the correct Pylon URL for the 'Contact support' node
2. Address 0% coverage in src/views/SupportTree.ts
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
No description provided.