`show-names` feature broken

❌ show-names
content.js:1880 Error: GraphQL:
Parse error on "[" (LBRACKET) at [1, 61]
    at content.js:1992

https://github.com/highlightjs/highlight.js/pull/2023

[jmilas]

Thanks for the details @bfred-it. We’ll get that fixed.

[notlmn]

Interesting. The bot has a link but:

• the link is a 404
• the hovercard also fails to load

That’s an issue from GitHub, the bot has a valid username (greenkeeper), but GH mistakenly appends [bot] to the link. GH was only supposed to append it to the text inside link, not the href of the anchor tag too (https://github.com/greenkeeper is a valid link).

/cc @lukehefson @shayfrendt

---

• skip names with [ (they’re bots)

How about removing that while querying, for now. Can be documented as a case for bot names.

---

• don’t add names to commits, which I think is duplicate info anyway 🤔

Yup agree, emails always exists, names may not (can be empty).