Skip to content

Add badge-11 through badge-15 inputs#28

Merged
StevenMilne-Sonocent merged 3 commits into
StevenMilneDev:mainfrom
genio-learn:add-badges-11-to-15
Jul 14, 2026
Merged

Add badge-11 through badge-15 inputs#28
StevenMilne-Sonocent merged 3 commits into
StevenMilneDev:mainfrom
genio-learn:add-badges-11-to-15

Conversation

@benmepham-genio

@benmepham-genio benmepham-genio commented Jun 5, 2026

Copy link
Copy Markdown
Contributor
  • Lifts the badge cap from 10 to 15 - we have some use cases where more than 10 is useful.
  • I updated the package-lock.json to point to npmjs rather than your private registry - up to you if you keep that or not!

Increase the badge cap from 10 to 15 so workflows declaring more
than 10 badges don't have their extras silently dropped by GitHub
Actions with "Unexpected input(s) 'badge-11'" warnings.

- action.yml: add badge-11..badge-15 (all required: false)
- src/util/actions.ts: extract MAX_BADGES = 15 and loop to it
- src/util/actions.test.ts: assert MAX_BADGES inputs are returned
  and extend the partial-fill test to cover the new range

The bin/index.js bundle needs regenerating (npm run build) before
this change takes effect at runtime — handled in a follow-up commit.
@benmepham-genio benmepham-genio marked this pull request as draft June 5, 2026 15:13
Two related fixes packaged together so `npm ci` works from a clean
checkout and the runtime picks up the badge-11..badge-15 inputs.

- package-lock.json: rewrite the 66 `resolved` URLs that point at
  https://nexus.smilne.dev/repository/npm-group/ (the maintainer's
  private Nexus mirror) to https://registry.npmjs.org/. Integrity
  hashes unchanged — Nexus was proxying npmjs.org so tarball
  contents are byte-identical.

- bin/index.js: regenerated via `npm run build` so the bundle
  actually exposes the MAX_BADGES = 15 loop. Without this, the
  preceding action.yml + actions.ts changes have no runtime effect.
@benmepham-genio benmepham-genio marked this pull request as ready for review June 5, 2026 15:27

@StevenMilne-Sonocent StevenMilne-Sonocent left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 😁

@StevenMilne-Sonocent StevenMilne-Sonocent merged commit e2676e9 into StevenMilneDev:main Jul 14, 2026
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