Skip to content

backport_pr.py: increase entries per page for branches GET#19985

Merged
bors[bot] merged 1 commit intoRIOT-OS:masterfrom
miri64:backport_pr/fix/fix-page-wrap
Oct 18, 2023
Merged

backport_pr.py: increase entries per page for branches GET#19985
bors[bot] merged 1 commit intoRIOT-OS:masterfrom
miri64:backport_pr/fix/fix-page-wrap

Conversation

@miri64
Copy link
Copy Markdown
Member

@miri64 miri64 commented Oct 18, 2023

Contribution description

More of a quick-and-dirty hack than an actual fix, but it should give us some time to properly fix it (I set the per_page to 100, currently there are 36 returned with that ;-))

Testing procedure

Try backporting a PR with it ;-).

Issues/PRs references

#19984 was created using this fix.

@miri64 miri64 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs labels Oct 18, 2023
@miri64 miri64 requested a review from MrKevinWeiss October 18, 2023 12:15
@github-actions github-actions bot added the Area: tools Area: Supplementary tools label Oct 18, 2023
@riot-ci
Copy link
Copy Markdown

riot-ci commented Oct 18, 2023

Murdock results

✔️ PASSED

25917c4 backport_pr.py: increase entries per page for branches GET

Success Failures Total Runtime
1 0 1 54s

Artifacts

Copy link
Copy Markdown
Contributor

@MrKevinWeiss MrKevinWeiss left a comment

Choose a reason for hiding this comment

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

Fixed for now... Let's let future us worry about a fix. ACK.

$ ./dist/tools/backport_pr/backport_pr.py 19978
Fetching for commit: #19978: treewide: fix typos to make codespell happy
found 4452f0729bd081bd316d145b55db68b5288970e5 : boards/lsn50: change image

Either the wiki is down, or the image was removed from upstream. This
replaces it with the same image hosted on the zephyr doc. As a site
effect, code spell likes the new URL much more.
found edc43201dbf023ca86fd0d8875aeafaecda438ba : tree-wide: fix typos in doc and comments

This should not change any generated binary
found 50d5c69e3e56f36115195e48107822244b5a9f55 : dist/tools/codespell: add more false positives
found 50490ed71b5aba19f771f9a1b2053b32a2d5b455 : dist/tools/lpc2k_pgm: fix typo in GUI / readme
Backport based on branch 2023.10-branch
Fetching upstream remote
Pushing branch backport/2023.10/fix-typos to origin
Pruning temporary workdir at /tmp/RIOT/backport_temp
Create PR number #19986 for backport

@MrKevinWeiss
Copy link
Copy Markdown
Contributor

bors merge

@bors
Copy link
Copy Markdown
Contributor

bors bot commented Oct 18, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 14b608c into RIOT-OS:master Oct 18, 2023
@miri64 miri64 deleted the backport_pr/fix/fix-page-wrap branch October 19, 2023 07:56
@MrKevinWeiss MrKevinWeiss added this to the Release 2024.01 milestone Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants