FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(federation): resolve analytics lint errors by mySebbe · Pull Request #14703 · NodeBB/NodeBB · GitHub

/ NodeBB Public

fix(federation): resolve analytics lint errors - #14703

Open
mySebbe wants to merge 1 commit into
NodeBB:developfrom
mySebbe:codex/fix-develop-federation-eslint-20260828
Open

fix(federation): resolve analytics lint errors#14703
mySebbe wants to merge 1 commit into
NodeBB:developfrom
mySebbe:codex/fix-develop-federation-eslint-20260828

Conversation

mySebbe commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Current develop CI currently stops at three ESLint errors introduced by the recent federation analytics changes:

  • ignore the unused map entry in the non-zero filter
  • generate pie-chart colors from entry keys without an unused count binding
  • apply the repository's arrow-parens style in the controller

This is intentionally limited to lint-only changes. Full ESLint completes with zero errors; the three existing no-await-in-loop warnings in src/activitypub/send.js remain unchanged.

This also unblocks the current develop workflow and the otherwise isolated ActivityPub test PR #14584.

Copy link
Copy Markdown
Contributor

💡 Friendly Note

This pull request was made against the develop branch, which is reserved for commits destined for a minor or major release.If your commits simply fixes a bug, please rebase this PR against the master instead.

  • patch releases — bug fixes only
  • minor releases — new features, enhancements, and bug fixes
  • major releases — breaking changes, including all of the above

Thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL