Zeke Sikelianos

localized number formatting

1

I noticed a module called `add-commas` in the codebase that may not be needed. Browsers already have a method called [Number.prototype.toLocaleString()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number/toLocaleString) which formats numbers according to the user's locale, i.e....

Unable to fetch repositories

1

Some fellow githubbers were calling me out for not having `LICENSE` files in some of my repos. Apparently indicating the license in `package.json` and `README.md` is not enough. So I...

Add a `rank` property to each emoji object.

2

👋 @mroth would you open to adding a `rank` or `ranking` property using the data from http://emojitracker.com/api/rankings? It could be added at build time by hitting the webservice directly, or...

Does it work with GitHub Actions?

2

Hi @JasonEtco! 👋 I see this in the README: > ci-reporter currently supports TravisCI and CircleCI. I'm working on a project that is now using GitHub Actions to run its...

hide comments when new commits are added to PR

1

As a PR progresses and the tests start to pass again, I often go back and manually hide old @ci-reporter comments. It would be cool if the bot could do...

What is the behavior for numerous failing builds?

3

Hi! I think we should give this bot a go on electron/electron org, but would first like to understand what it does when it encounters multiple failing builds, like: ...

Similar to #262, but this would just make reviewing PRs a bit easier.

Revamping the release process

2

@codebytere and I cut a new release today! https://github.com/electron/electron-api-demos/releases/tag/v1.4.0 Here's some followup work we need to do: - decommission old update server - remove update server deployment stuff from demo...

Preserve some HTML from descriptions

1

We want the `WebContents` and the link here. Not just the raw text. https://github.com/zeke/electron-docs-linter/blob/b5a9608f4cbfbff9e32ec4c5771fe488d24d9c2b/lib/helpers.js#L19 

For example, https://github.com/electron/electron/blob/master/docs/api/app.md#apprelaunchoptions is composed of several paragraphs. ```js const apis = require('electron-api-docs/tree') console.log(apis.app.methods.relaunch.description) ``` currently produces ``` Relaunches the app when current instance exits. By default the new instance...