Draft
Conversation
Made-with: Cursor
Made-with: Cursor
- Enhanced workflow with test file counts and coverage breakdown in GitHub Actions summary - Added codecov.yml with unit test flag and component definitions - Added title to CTRF reporter for clearer test result labeling Made-with: Cursor
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly enhances the project's quality assurance metrics by establishing robust code coverage tracking and standardizing test result reporting. By integrating Codecov and adopting the Common Test Report Format, the changes provide clearer insights into code quality and test execution, facilitating better analysis and continuous improvement of the codebase. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Ignored Files
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request enhances QA metrics by adding Codecov configuration and a CTRF test reporter. While no security vulnerabilities were found, my review identified areas for improvement. The Codecov configuration could be strengthened by requiring CI to pass and setting a more meaningful coverage threshold. Critically, a generated test report file containing absolute local file paths has been added to the repository; this file should be removed and added to .gitignore to prevent leaking sensitive information and repository bloat.
Comment on lines
+1
to
+492
| { | ||
| "results": { | ||
| "tool": { | ||
| "name": "mocha" | ||
| }, | ||
| "summary": { | ||
| "tests": 43, | ||
| "passed": 43, | ||
| "failed": 0, | ||
| "pending": 0, | ||
| "skipped": 0, | ||
| "other": 0, | ||
| "start": 1772833660899, | ||
| "stop": 1772833660918 | ||
| }, | ||
| "tests": [ | ||
| { | ||
| "name": "file exists resolves if the file exists", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/file.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660900, | ||
| "stop": 1772833660900 | ||
| }, | ||
| { | ||
| "name": "file exists rejects if the file does not exist", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/file.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660901, | ||
| "stop": 1772833660901 | ||
| }, | ||
| { | ||
| "name": "file readIfExists returns the contents of the file if it exists", | ||
| "status": "passed", | ||
| "duration": 1, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/file.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660901, | ||
| "stop": 1772833660902 | ||
| }, | ||
| { | ||
| "name": "file readIfExists returns the empty string if it does not exist", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/file.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660902, | ||
| "stop": 1772833660902 | ||
| }, | ||
| { | ||
| "name": "file writeToFile writes to the file if the path is a normal path", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/file.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660903, | ||
| "stop": 1772833660903 | ||
| }, | ||
| { | ||
| "name": "file writeToFile writes to stdout if path is \"-\"", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/file.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660903, | ||
| "stop": 1772833660903 | ||
| }, | ||
| { | ||
| "name": "git getCommits passes in revisions if there is a previous tag", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/git.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660904, | ||
| "stop": 1772833660904 | ||
| }, | ||
| { | ||
| "name": "git getCommits does not pass in revisions if there are no previous tags", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/git.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660905, | ||
| "stop": 1772833660905 | ||
| }, | ||
| { | ||
| "name": "git getCommits uses custom revision range if `-t` / `--tag` option was used", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/git.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660905, | ||
| "stop": 1772833660905 | ||
| }, | ||
| { | ||
| "name": "git getCommits errors if there are no commits yet", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/git.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660906, | ||
| "stop": 1772833660906 | ||
| }, | ||
| { | ||
| "name": "git getCommits correctly parses type, category, and subject", | ||
| "status": "passed", | ||
| "duration": 1, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/git.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660907, | ||
| "stop": 1772833660908 | ||
| }, | ||
| { | ||
| "name": "git getCommits correctly applies flags [breaking]", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/git.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660908, | ||
| "stop": 1772833660908 | ||
| }, | ||
| { | ||
| "name": "git getCommits skips malformed commits", | ||
| "status": "passed", | ||
| "duration": 1, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/git.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660908, | ||
| "stop": 1772833660909 | ||
| }, | ||
| { | ||
| "name": "git getCommits skips any excluded commit types", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/git.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660909, | ||
| "stop": 1772833660909 | ||
| }, | ||
| { | ||
| "name": "package getUserPackage pull the package.json from the current directory", | ||
| "status": "passed", | ||
| "duration": 1, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660909, | ||
| "stop": 1772833660910 | ||
| }, | ||
| { | ||
| "name": "package getUserPackage errs if there is no package.json", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660910, | ||
| "stop": 1772833660910 | ||
| }, | ||
| { | ||
| "name": "package getUserPackage errs if the package.json is invalid", | ||
| "status": "passed", | ||
| "duration": 1, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660910, | ||
| "stop": 1772833660911 | ||
| }, | ||
| { | ||
| "name": "package extractRepoUrl returns null if there is no repo URL", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660911, | ||
| "stop": 1772833660911 | ||
| }, | ||
| { | ||
| "name": "package extractRepoUrl returns the raw URL if it is not a GitHub URL", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660911, | ||
| "stop": 1772833660911 | ||
| }, | ||
| { | ||
| "name": "package extractRepoUrl correctly parses a GitHub URL", | ||
| "status": "passed", | ||
| "duration": 1, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660911, | ||
| "stop": 1772833660912 | ||
| }, | ||
| { | ||
| "name": "package calculateNewVersion bumps the major version if major is true", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660912, | ||
| "stop": 1772833660912 | ||
| }, | ||
| { | ||
| "name": "package calculateNewVersion bumps the minor version if minor is true", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660912, | ||
| "stop": 1772833660912 | ||
| }, | ||
| { | ||
| "name": "package calculateNewVersion bumps the patch version if patch is true", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660912, | ||
| "stop": 1772833660912 | ||
| }, | ||
| { | ||
| "name": "package calculateNewVersion leaves the version untouched if none of three options is true", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660912, | ||
| "stop": 1772833660912 | ||
| }, | ||
| { | ||
| "name": "package calculateNewVersion returns null if no version is specified", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/package.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660913, | ||
| "stop": 1772833660913 | ||
| }, | ||
| { | ||
| "name": "writer getCommitUrl makes a valid URL for a BitBucket repository", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660913, | ||
| "stop": 1772833660913 | ||
| }, | ||
| { | ||
| "name": "writer getCommitUrl makes a valid URL for a GitHub repository", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660913, | ||
| "stop": 1772833660913 | ||
| }, | ||
| { | ||
| "name": "writer getCommitUrl makes a valid URL for a Gitlab repository", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660913, | ||
| "stop": 1772833660913 | ||
| }, | ||
| { | ||
| "name": "writer markdown makes heading h2 if major version", | ||
| "status": "passed", | ||
| "duration": 2, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660913, | ||
| "stop": 1772833660915 | ||
| }, | ||
| { | ||
| "name": "writer markdown makes heading h3 if minor version", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660915, | ||
| "stop": 1772833660915 | ||
| }, | ||
| { | ||
| "name": "writer markdown makes heading h4 if minor version", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660915, | ||
| "stop": 1772833660915 | ||
| }, | ||
| { | ||
| "name": "writer markdown keeps only the date if no version is specified", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660915, | ||
| "stop": 1772833660915 | ||
| }, | ||
| { | ||
| "name": "writer markdown flushes out a commit type with its full name", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660916, | ||
| "stop": 1772833660916 | ||
| }, | ||
| { | ||
| "name": "writer markdown uses the default type name for uncommon types", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660916, | ||
| "stop": 1772833660916 | ||
| }, | ||
| { | ||
| "name": "writer markdown groups all uncommon types together", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660916, | ||
| "stop": 1772833660916 | ||
| }, | ||
| { | ||
| "name": "writer markdown does not group uncommon types if unknown types are allowed", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660916, | ||
| "stop": 1772833660916 | ||
| }, | ||
| { | ||
| "name": "writer markdown keeps a commit category on one line if there is only one commit in it", | ||
| "status": "passed", | ||
| "duration": 1, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660916, | ||
| "stop": 1772833660917 | ||
| }, | ||
| { | ||
| "name": "writer markdown breaks a commit category onto its own line if there is more than one commit in it", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660917, | ||
| "stop": 1772833660917 | ||
| }, | ||
| { | ||
| "name": "writer markdown omits commit category if there was no category defined", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660917, | ||
| "stop": 1772833660917 | ||
| }, | ||
| { | ||
| "name": "writer markdown trims the commit hash to only 8 chars", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660917, | ||
| "stop": 1772833660917 | ||
| }, | ||
| { | ||
| "name": "writer markdown wraps the hash in a link if a repoUrl is provided", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660917, | ||
| "stop": 1772833660917 | ||
| }, | ||
| { | ||
| "name": "writer markdown wraps an issue/pr number if a repoUrl is provided", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660918, | ||
| "stop": 1772833660918 | ||
| }, | ||
| { | ||
| "name": "writer markdown wraps more than one issue/pr numbers in one commit if a repoUrl is provided", | ||
| "status": "passed", | ||
| "duration": 0, | ||
| "retries": 0, | ||
| "flaky": false, | ||
| "filePath": "/Users/vikita.bhandari/Documents/qa-coverage/repos/generate-changelog/test/writer.test.js", | ||
| "rawStatus": "passed", | ||
| "start": 1772833660918, | ||
| "stop": 1772833660918 | ||
| } | ||
| ] | ||
| } | ||
| } |
There was a problem hiding this comment.
This file appears to be a generated test report. Committing generated files to the repository is generally discouraged as it can lead to merge conflicts and bloat the repository size.
More importantly, this file contains:
- Absolute file paths (e.g.,
"/Users/vikita.bhandari/..."), which is a security risk as it leaks information about the local development environment. - Timestamps that are far in the future (e.g.,
1772833660899corresponds to the year 2215), which might indicate an issue with the test reporter or its configuration.
This file should be removed from version control, and the ctrf/ directory should be added to your .gitignore file.
| @@ -0,0 +1,31 @@ | |||
| codecov: | |||
| require_ci_to_pass: false | |||
There was a problem hiding this comment.
Setting require_ci_to_pass to false is not recommended. It means that Codecov will process coverage reports even if the CI build fails, which could lead to misleading coverage data if tests are not running successfully. It's better to ensure the CI pipeline passes before analyzing coverage.
require_ci_to_pass: true| project: | ||
| default: | ||
| target: auto | ||
| threshold: 1% |
There was a problem hiding this comment.
A coverage threshold of 1% is very low and does not provide a meaningful quality gate. While this might be for initial setup, it's recommended to set a more ambitious target to ensure that code coverage doesn't degrade. Consider increasing this to a higher value, for example 5%.
threshold: 5%- Install mocha-junit-reporter to generate JUnit XML test results - Upload JUnit XML to Codecov via test-results-action with unit flag - Adds test health metrics (flaky detection, failure rates, durations) to Codecov dashboard - Add test-results/ to .gitignore Made-with: Cursor
Made-with: Cursor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.