[CI][Docs] Pull requests closed by the PR limit workflow cannot be reopened by their authors
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 68/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- github-actions, javascript, markdown
- Domain
- ci-cd, documentation
Research direction
Read .github/workflows/pr_limit/comment.md, docs/source/developers/bug_reports.rst, and .github/workflows/pr_limit/check.js to understand the conflicting reopen instructions and current workflow behavior. Decide whether to revise the guidance or workflow, then verify authors below the PR limit can follow the documented path to reopen a closed pull request.
Written by the indexing model from the issue text.
Description
Describe the bug, including details regarding any error messages, version, and platform.
The PR limit workflow closes a pull request that takes a contributor over the limit, and both its comment (.github/workflows/pr_limit/comment.md: "you are welcome to reopen this one") and the docs (docs/source/developers/bug_reports.rst: "Once one of your other pull requests has been merged or closed, you can reopen it") say that the author can reopen it later.
In practice the author cannot. After #54656 was merged, which brought me below the limit, reopening #54657 (closed by the workflow) fails:
$ gh pr reopen 54657 -R apache/arrow
API call failed: GraphQL: Could not open the pull request. (reopenPullRequest)
Presumably GitHub does not let an author without repository access reopen a pull request that was closed by an account with write access.
Either the comment and the docs could ask the author to request a reopen from a committer or collaborator (the workflow already leaves alone a pull request reopened by someone with repository access, see .github/workflows/pr_limit/check.js), or the workflow could let authors reopen their pull requests, e.g. by reopening them itself once the author is below the limit.
Component(s)
Continuous Integration, Documentation
- Dominant language
- C++
- Stars
- 17.2k
- Forks
- 4.4k
- Avg merge
- 2d 14h
- Merged PRs (30d)
- 70
Getting set up
- Ships a Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from apache/arrow
-
[R][Parquet] ParquetReaderProperties does not expose page_checksum_verification, so a corrupt page CRC reads silentlyMay be free again A pull request for this issue was closed without being merged. OpenComponent: R good-first-issue Type: enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
apache/arrow#53138 · 2 comments ·
Maintainers usually reply within 1 day
-
[C++][Python] IPC reader rejects a DictionaryEncoding without indexType, which the format allowsPossibly taken A pull request linked to this issue is open or already merged. OpenComponent: C++ Component: Python Type: bug
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
[R] Silent loss of metadata integrity for date-time and numeric attributes in write_parquet()/read_parquet()May be free again @james-finn-travers claimed this 26 days ago, and no pull request is open. OpenComponent: R Type: bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
apache/arrow#51695 · 1 comment ·
Maintainers usually reply within 1 day
-
[C++][Parquet] Plaintext-footer files written with AES_GCM_CTR_V1 record AES_GCM_V1 as the encryption algorithm and cannot be readPossibly taken @autoCAT6 claimed this 7 days ago. OpenType: bug
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
Maintainers usually reply within 1 day
-
[R] Expose ignore_extra_columns and pad_short_rows CSV parse optionsPossibly taken A pull request linked to this issue is open or already merged. OpenComponent: R good-first-issue Type: enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
apache/arrow#51659 · 1 comment ·
Maintainers usually reply within 1 day
Similar issues
-
Component - Build
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day
-
bug-unconfirmed
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
wazuh/wazuh#40870 · 1 comment ·
Maintainers usually reply within 1 day
-
documentation
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
tenstorrent/tt-metal#63198 ·
Maintainers usually reply within 1 day
-
Extended search mode does not find escape sequences (\t, \n, \r...)Possibly taken @CyrilB1531 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 87/100
dail8859/NotepadNext#1181 ·
Maintainers usually reply within 1 day