Skip to content

Add auto-populated authors#4238

Merged
gfwilliams merged 6 commits into
espruino:masterfrom
RKBoss6:authors
Jun 2, 2026
Merged

Add auto-populated authors#4238
gfwilliams merged 6 commits into
espruino:masterfrom
RKBoss6:authors

Conversation

@RKBoss6
Copy link
Copy Markdown
Contributor

@RKBoss6 RKBoss6 commented Jun 1, 2026

Adds auto-populated authors by using the first commit author of each app. Some apps have an author who's username was changed, or the account was deleted, so those apps were ignored and unchanged. More info can be found in the discussion

Copilot AI review requested due to automatic review settings June 1, 2026 11:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@gfwilliams
Copy link
Copy Markdown
Member

As we're doing this, please could you try and get the author field a bit futher up the JSON rather than right at the end?

When @bobrippling did this for https://github.com/espruino/BangleApps/pull/3994/changes#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5 he managed to get it after version/before description which looks good.

There's the script he used in there which does it already, so we could use that (I think the issue was just GitHub API rate limiting?)

@RKBoss6
Copy link
Copy Markdown
Contributor Author

RKBoss6 commented Jun 2, 2026

Should be good to go now!

@gfwilliams
Copy link
Copy Markdown
Member

Great - thanks!

@gfwilliams gfwilliams merged commit b0f5c92 into espruino:master Jun 2, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants