My experience of GitRoll

What is GitRoll?

A skill assessment tool for GitHub repositories.

The link: GitRoll

How I use GitRoll?

  1. Go to the Home page and click on Scan a GitHub.

  2. Enter my GitHub repository URL and click on Start.

  3. After the scan is done, I can see my skill overview page.

  4. Click the title of a result card to see the details. Let’s take a look at this failed card.

  5. The detail page shows information including: Repository Name and URL, Lines of Code, Stars, Last Updated Time, Languages, Skills, Reliability, Security, Maintainability, Issues.

  6. Click one of Bug issue: Unexpected duplicate "position". We can see the number of line is 330. This would cause unexpected layout of an HTML element.

  7. See More Cases for more.

More Cases

  1. AsherJingkongChen/biosemo
    1. Bug - High: Remove all characters located before "<?xml".
      • According to the XML specification, this is not a valid XML file. This issue is correct. However, this severity should not be High. Some implementations may ignore the characters before "<?xml" and parse the XML file correctly. This issue should be Medium. Furthermore, this XML file is not crucial to the project.

    2. Smell - Medium: Remove this commented out code.
      • This is needed in practice.

  2. AsherJingkongChen/auto-media-type
    1. Smell - Low: Expected a for-of loop instead of a for loop with this simple iteration.
      • Better to use for loop for higher throughput. See this benchmark result: for-of vs for.

  3. AsherJingkongChen/xp-web
    1. Smell - Medium: Unnecessary escape character: \/.
      • Yes, it is unnecessary and has no effect.

    2. Bug - Medium: Add a <title> tag to this page.
      • According to the HTML specification, the title element is not omissible. However, this file is the HTML template for Svelte, a web framework like Vue. A <title> tag will be interpolated by Svelte in SSR/SSG phase.

  4. AsherJingkongChen/hello-world-grpc-web
    1. Smell - Low/Medium: Remove the unused parameter/variable.
      • Generated codes by protoc are not perfect. This is a common issue.

    2. Smell - High: Unexpected var, use let or const instead.
      • A generated codes common issue.

  5. AsherJingkongChen/codeslide
    1. Smell - Low: Prefer using nullish coalescing operator (??) instead of a logical or (||), as it is a safer operator.
      • This is needed in practice. We need to make "" || "plaintext" to return "plaintext".

  6. antfu/vitesse
    1. Bug - Medium: Add a <title> tag to this page.
      • According to the HTML specification, the title element is not omissible. However, this file is the HTML template for Vue, a web framework like Angular. A <title> tag will be interpolated by Vue in SSG phase.

  7. antfu/vite-ssg
    1. Bug - Medium: Expected an assignment or function call and instead saw an expression.
      • This is the API design of yargs (Global builder), a CLI argument parser. On the other hand, there is an eslint-disable banner on the first line, which tells eslint to ignore any unused expression.

  8. 101arrowz/fflate
    1. Bug - Medium: Remove this use of the output from "wbytes"; "wbytes" doesn’t return anything.
      • This is neither a bug nor a smell. This function returns void, and its output is not used. This is a false issue.

    2. Bug - High: 'l' is not defined in this loop.
      • l is defined in the outer scope (the fifth parameter of e). This is a false issue.

    3. Smell - High: Unexpected var, use let or const instead. (xN)
      • A generated codes common issue.

Conclusion

It is an AI tool. It is not perfect but good enough to use.

Reference

NOTE: Links may be invalid in the future. I do not have any archive of them

  1. GitRoll
  2. GitRoll - AsherJingkongChen
  3. GitRoll - antfu
  4. GitRoll - 101arrowz

Declaimer

All the information is public. These documents and images are assets of Public Domain.