actions/toolkit
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
1,783 Commits 271 Branches 19 Tags
danwkennedy/bump-http-client-github
Commit Graph
6 Commits
This Branch
This Branch
All Branches
Author SHA1 Message Date
Austen Stone 006d6978c1 linting 2025-10-22 11:44:21 -04:00
Austen Stone 02afeb1577 style: wrap ACTIONS_ARTIFACT_MAX_ARTIFACT_COUNT env var assignment for readability 2025-10-22 11:42:01 -04:00
Austen Stone cbc06d6766 fix: ensure max artifact count variable is treated as a string 2025-10-22 07:47:51 -04:00
Austen Stone 9bb6708527 fix: remove redundant check for max artifact count variable 2025-10-22 07:41:31 -04:00
Austen StoneGitHubBassem Dghaidi
 		be1151df02 Apply suggestion from @Link- 
Co-authored-by: Bassem Dghaidi <568794+Link-@users.noreply.github.com>
 2025-10-22 07:39:45 -04:00
Austen Stone fea4f6b5c5 fix: resolve critical pagination bugs and add comprehensive testing 
- Fix off-by-one error in pagination loop (< to <=) that prevented fetching last page
- Add Math.ceil() to maxNumberOfPages calculation for proper limit handling
- Replace hardcoded 2000 limit with configurable getMaxArtifactListCount()
- Add pagination test for multi-page artifact listing
- Add environment variable test for ACTIONS_ARTIFACT_MAX_ARTIFACT_COUNT
- Add comprehensive test coverage for getMaxArtifactListCount() function

Fixes compound bug where pagination and limit logic capped results at 900 artifacts instead of intended 1000.
 2025-10-21 09:22:11 -04:00