[PR #7309] [MERGED] count points script should fetch the release note from the PR directly #21439

Closed
opened 2026-04-14 22:04:43 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/7309
Author: @matt-fidd
Created: 3/27/2026
Status: Merged
Merged: 3/30/2026
Merged by: @matt-fidd

Base: masterHead: points-pr-category


📝 Commits (3)

  • 41eefe3 get pr release note from PR, not top of master
  • bf8f146 note
  • a5b8e6f Merge branch 'master' into points-pr-category

📊 Changes

2 files changed (+22 additions, -62 deletions)

View changed files

📝 .github/scripts/count-points.mjs (+16 -62)
upcoming-release-notes/7309.md (+6 -0)

📄 Description

Description

PRs merged in the current month but before the last release had their release note file removed from master as part of the release process, meaning it could not be fetched during the next month's points count run. Instead, we should just grab the release note from the PR itself.

As a bonus, it saves us some API requests.

Found while testing https://github.com/actualbudget/actual/pull/7305

Testing

Before (notice the unknown for 7034):

GET /repos/actualbudget/actual/contents/upcoming-release-notes%2F7043.md?ref=eabf09587f04ec5142f16e1c33086a6e89f91f60 - 404 with id F7FA:6CC12:24C4B7:2DEB1D:69C711D5 in 145ms
GET /repos/actualbudget/actual/contents/upcoming-release-notes%2F6975.md?ref=eabf09587f04ec5142f16e1c33086a6e89f91f60 - 404 with id F803:2C9C4D:2957BF:327ED0:69C711D5 in 142ms
GET /repos/actualbudget/actual/contents/upcoming-release-notes%2F7034.md?ref=eabf09587f04ec5142f16e1c33086a6e89f91f60 - 404 with id F7FE:83C97:222336:2B47A6:69C711D5 in 135ms
GET /repos/actualbudget/actual/contents/upcoming-release-notes%2F7040.md?ref=eabf09587f04ec5142f16e1c33086a6e89f91f60 - 404 with id F808:353EDF:27102D:3035F9:69C711D5 in 135ms
GET /repos/actualbudget/actual/contents/upcoming-release-notes%2F6976.md?ref=eabf09587f04ec5142f16e1c33086a6e89f91f60 - 404 with id F805:229929:2A2EC1:3354D8:69C711D6 in 147ms

...

PR Contribution Statistics:
===========================
MatissJanis: 57 (PRs: #7094 (2pts - Maintenance), #7081 (3pts - Bugfix), #7052 (2pts - Maintenance), #7084 (3pts - Bugfixes), #6983 (3pts - Bugfixes), #7043 (2pts - Unknown), #6984 (2pts - Maintenance), #7034 (2pts - Unknown), #6965 (2pts - Maintenance), #7040 (2pts - Unknown), #6970 (2pts - Maintenance), #6964 (2pts - Maintenance), #7019 (2pts - Maintenance), #7018 (2pts - Enhancements), #7033 (2pts - Maintenance), #6955 (2pts - Maintenance), #6985 (2pts - Maintenance), #6963 (2pts - Maintenance), #6989 (2pts - Maintenance), #7088 (2pts - Maintenance), #6987 (2pts - Maintenance), #6796 (2pts - Maintenance), #6721 (2pts - Maintenance), #6784 (2pts - Maintenance), #6801 (2pts - Maintenance), #6481 (2pts - Maintenance), #6805 (2pts - Maintenance))
joel-jeremy: 47 (PRs: #6958 (2pts - Enhancements), #7009 (2pts - Maintenance), #6957 (2pts - Enhancements), #7007 (3pts - Bugfixes), #6951 (2pts - Maintenance), #6953 (3pts - Bugfixes), #6942 (3pts - Bugfixes), #6941 (2pts - Enhancements), #6936 (2pts - Maintenance), #6883 (2pts - Maintenance), #6861 (2pts - Maintenance), #6867 (2pts - Maintenance), #6862 (2pts - Maintenance), #6864 (2pts - Maintenance), #6868 (2pts - Maintenance), #6882 (2pts - Maintenance), #6880 (2pts - Enhancements), #6863 (2pts - Maintenance), #6757 (2pts - Maintenance), #6943 (2pts - Maintenance), #6140 (2pts - Maintenance), #5977 (2pts - Maintenance))
jfdoming: 11 (PRs: #7067 (3pts - Bugfixes), #6992 (2pts - Maintenance), #6969 (2pts - Maintenance), #6692 (2pts - Enhancements), #6690 (2pts - Enhancements))
MikesGlitch: 11 (PRs: #7076 (3pts - Bugfix), #6940 (2pts - Maintenance), #6924 (2pts - Maintenance), #6910 (2pts - Maintenance), #6874 (2pts - Maintenance))
matt-fidd: 8 (PRs: #7013 (2pts - Maintenance), #7021 (2pts - Maintenance), #7022 (2pts - Maintenance), #6840 (2pts - Maintenance))
youngcw: 7 (PRs: #6975 (2pts - Unknown), #6976 (2pts - Unknown), #6922 (3pts - Bugfixes))

After:

PR Contribution Statistics:
===========================
MatissJanis: 59 (PRs: #7094 (2pts - Maintenance), #7052 (2pts - Maintenance), #7084 (3pts - Bugfixes), #7034 (3pts - Bugfixes), #6985 (2pts - Maintenance), #7088 (2pts - Maintenance), #6970 (2pts - Maintenance), #7043 (2pts - Unknown), #6964 (2pts - Maintenance), #6963 (2pts - Maintenance), #7033 (2pts - Maintenance), #7081 (3pts - Bugfix), #6984 (2pts - Maintenance), #6955 (2pts - Maintenance), #6965 (2pts - Maintenance), #6989 (2pts - Maintenance), #7018 (2pts - Enhancements), #6983 (3pts - Bugfixes), #7019 (2pts - Maintenance), #7040 (3pts - Bugfixes), #6796 (2pts - Maintenance), #6987 (2pts - Maintenance), #6721 (2pts - Maintenance), #6801 (2pts - Maintenance), #6784 (2pts - Maintenance), #6481 (2pts - Maintenance), #6805 (2pts - Maintenance))
joel-jeremy: 47 (PRs: #7007 (3pts - Bugfixes), #6958 (2pts - Enhancements), #7009 (2pts - Maintenance), #6957 (2pts - Enhancements), #6942 (3pts - Bugfixes), #6953 (3pts - Bugfixes), #6951 (2pts - Maintenance), #6941 (2pts - Enhancements), #6936 (2pts - Maintenance), #6883 (2pts - Maintenance), #6943 (2pts - Maintenance), #6861 (2pts - Maintenance), #6882 (2pts - Maintenance), #6868 (2pts - Maintenance), #6864 (2pts - Maintenance), #6862 (2pts - Maintenance), #6863 (2pts - Maintenance), #6867 (2pts - Maintenance), #6880 (2pts - Enhancements), #6757 (2pts - Maintenance), #6140 (2pts - Maintenance), #5977 (2pts - Maintenance))
jfdoming: 11 (PRs: #7067 (3pts - Bugfixes), #6992 (2pts - Maintenance), #6969 (2pts - Maintenance), #6692 (2pts - Enhancements), #6690 (2pts - Enhancements))
MikesGlitch: 11 (PRs: #7076 (3pts - Bugfix), #6940 (2pts - Maintenance), #6924 (2pts - Maintenance), #6910 (2pts - Maintenance), #6874 (2pts - Maintenance))
matt-fidd: 8 (PRs: #7013 (2pts - Maintenance), #7021 (2pts - Maintenance), #7022 (2pts - Maintenance), #6840 (2pts - Maintenance))
youngcw: 7 (PRs: #6975 (2pts - Unknown), #6976 (2pts - Unknown), #6922 (3pts - Bugfixes))

Checklist

  • Release notes added (see link above)
  • No obvious regressions in affected areas
  • Self-review has been performed - I understand what each change in the code does and why it is needed

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/7309 **Author:** [@matt-fidd](https://github.com/matt-fidd) **Created:** 3/27/2026 **Status:** ✅ Merged **Merged:** 3/30/2026 **Merged by:** [@matt-fidd](https://github.com/matt-fidd) **Base:** `master` ← **Head:** `points-pr-category` --- ### 📝 Commits (3) - [`41eefe3`](https://github.com/actualbudget/actual/commit/41eefe3ae9175a644def5d73b96892bfc832675c) get pr release note from PR, not top of master - [`bf8f146`](https://github.com/actualbudget/actual/commit/bf8f146cc8f6b7239397a7e708233bd78b55a40e) note - [`a5b8e6f`](https://github.com/actualbudget/actual/commit/a5b8e6f87adb5da4e65f7bd68f5754f411db65a1) Merge branch 'master' into points-pr-category ### 📊 Changes **2 files changed** (+22 additions, -62 deletions) <details> <summary>View changed files</summary> 📝 `.github/scripts/count-points.mjs` (+16 -62) ➕ `upcoming-release-notes/7309.md` (+6 -0) </details> ### 📄 Description <!-- Thank you for submitting a pull request! Make sure to follow the instructions to write release notes for your PR — it should only take a minute or two: https://github.com/actualbudget/docs#writing-good-release-notes. Try running yarn generate:release-notes *before* pushing your PR for an interactive experience. --> ## Description <!-- What does this PR do? Why is it needed? Please give context on the "why?": why do we need this change? What problem is it solving for you?--> PRs merged in the current month but before the last release had their release note file removed from master as part of the release process, meaning it could not be fetched during the next month's points count run. Instead, we should just grab the release note from the PR itself. As a bonus, it saves us some API requests. Found while testing https://github.com/actualbudget/actual/pull/7305 ## Testing <!-- What did you test? How can we reproduce the issue you are fixing or how can we test the feature you built? --> Before (notice the unknown for 7034): ``` GET /repos/actualbudget/actual/contents/upcoming-release-notes%2F7043.md?ref=eabf09587f04ec5142f16e1c33086a6e89f91f60 - 404 with id F7FA:6CC12:24C4B7:2DEB1D:69C711D5 in 145ms GET /repos/actualbudget/actual/contents/upcoming-release-notes%2F6975.md?ref=eabf09587f04ec5142f16e1c33086a6e89f91f60 - 404 with id F803:2C9C4D:2957BF:327ED0:69C711D5 in 142ms GET /repos/actualbudget/actual/contents/upcoming-release-notes%2F7034.md?ref=eabf09587f04ec5142f16e1c33086a6e89f91f60 - 404 with id F7FE:83C97:222336:2B47A6:69C711D5 in 135ms GET /repos/actualbudget/actual/contents/upcoming-release-notes%2F7040.md?ref=eabf09587f04ec5142f16e1c33086a6e89f91f60 - 404 with id F808:353EDF:27102D:3035F9:69C711D5 in 135ms GET /repos/actualbudget/actual/contents/upcoming-release-notes%2F6976.md?ref=eabf09587f04ec5142f16e1c33086a6e89f91f60 - 404 with id F805:229929:2A2EC1:3354D8:69C711D6 in 147ms ... PR Contribution Statistics: =========================== MatissJanis: 57 (PRs: #7094 (2pts - Maintenance), #7081 (3pts - Bugfix), #7052 (2pts - Maintenance), #7084 (3pts - Bugfixes), #6983 (3pts - Bugfixes), #7043 (2pts - Unknown), #6984 (2pts - Maintenance), #7034 (2pts - Unknown), #6965 (2pts - Maintenance), #7040 (2pts - Unknown), #6970 (2pts - Maintenance), #6964 (2pts - Maintenance), #7019 (2pts - Maintenance), #7018 (2pts - Enhancements), #7033 (2pts - Maintenance), #6955 (2pts - Maintenance), #6985 (2pts - Maintenance), #6963 (2pts - Maintenance), #6989 (2pts - Maintenance), #7088 (2pts - Maintenance), #6987 (2pts - Maintenance), #6796 (2pts - Maintenance), #6721 (2pts - Maintenance), #6784 (2pts - Maintenance), #6801 (2pts - Maintenance), #6481 (2pts - Maintenance), #6805 (2pts - Maintenance)) joel-jeremy: 47 (PRs: #6958 (2pts - Enhancements), #7009 (2pts - Maintenance), #6957 (2pts - Enhancements), #7007 (3pts - Bugfixes), #6951 (2pts - Maintenance), #6953 (3pts - Bugfixes), #6942 (3pts - Bugfixes), #6941 (2pts - Enhancements), #6936 (2pts - Maintenance), #6883 (2pts - Maintenance), #6861 (2pts - Maintenance), #6867 (2pts - Maintenance), #6862 (2pts - Maintenance), #6864 (2pts - Maintenance), #6868 (2pts - Maintenance), #6882 (2pts - Maintenance), #6880 (2pts - Enhancements), #6863 (2pts - Maintenance), #6757 (2pts - Maintenance), #6943 (2pts - Maintenance), #6140 (2pts - Maintenance), #5977 (2pts - Maintenance)) jfdoming: 11 (PRs: #7067 (3pts - Bugfixes), #6992 (2pts - Maintenance), #6969 (2pts - Maintenance), #6692 (2pts - Enhancements), #6690 (2pts - Enhancements)) MikesGlitch: 11 (PRs: #7076 (3pts - Bugfix), #6940 (2pts - Maintenance), #6924 (2pts - Maintenance), #6910 (2pts - Maintenance), #6874 (2pts - Maintenance)) matt-fidd: 8 (PRs: #7013 (2pts - Maintenance), #7021 (2pts - Maintenance), #7022 (2pts - Maintenance), #6840 (2pts - Maintenance)) youngcw: 7 (PRs: #6975 (2pts - Unknown), #6976 (2pts - Unknown), #6922 (3pts - Bugfixes)) ``` After: ``` PR Contribution Statistics: =========================== MatissJanis: 59 (PRs: #7094 (2pts - Maintenance), #7052 (2pts - Maintenance), #7084 (3pts - Bugfixes), #7034 (3pts - Bugfixes), #6985 (2pts - Maintenance), #7088 (2pts - Maintenance), #6970 (2pts - Maintenance), #7043 (2pts - Unknown), #6964 (2pts - Maintenance), #6963 (2pts - Maintenance), #7033 (2pts - Maintenance), #7081 (3pts - Bugfix), #6984 (2pts - Maintenance), #6955 (2pts - Maintenance), #6965 (2pts - Maintenance), #6989 (2pts - Maintenance), #7018 (2pts - Enhancements), #6983 (3pts - Bugfixes), #7019 (2pts - Maintenance), #7040 (3pts - Bugfixes), #6796 (2pts - Maintenance), #6987 (2pts - Maintenance), #6721 (2pts - Maintenance), #6801 (2pts - Maintenance), #6784 (2pts - Maintenance), #6481 (2pts - Maintenance), #6805 (2pts - Maintenance)) joel-jeremy: 47 (PRs: #7007 (3pts - Bugfixes), #6958 (2pts - Enhancements), #7009 (2pts - Maintenance), #6957 (2pts - Enhancements), #6942 (3pts - Bugfixes), #6953 (3pts - Bugfixes), #6951 (2pts - Maintenance), #6941 (2pts - Enhancements), #6936 (2pts - Maintenance), #6883 (2pts - Maintenance), #6943 (2pts - Maintenance), #6861 (2pts - Maintenance), #6882 (2pts - Maintenance), #6868 (2pts - Maintenance), #6864 (2pts - Maintenance), #6862 (2pts - Maintenance), #6863 (2pts - Maintenance), #6867 (2pts - Maintenance), #6880 (2pts - Enhancements), #6757 (2pts - Maintenance), #6140 (2pts - Maintenance), #5977 (2pts - Maintenance)) jfdoming: 11 (PRs: #7067 (3pts - Bugfixes), #6992 (2pts - Maintenance), #6969 (2pts - Maintenance), #6692 (2pts - Enhancements), #6690 (2pts - Enhancements)) MikesGlitch: 11 (PRs: #7076 (3pts - Bugfix), #6940 (2pts - Maintenance), #6924 (2pts - Maintenance), #6910 (2pts - Maintenance), #6874 (2pts - Maintenance)) matt-fidd: 8 (PRs: #7013 (2pts - Maintenance), #7021 (2pts - Maintenance), #7022 (2pts - Maintenance), #6840 (2pts - Maintenance)) youngcw: 7 (PRs: #6975 (2pts - Unknown), #6976 (2pts - Unknown), #6922 (3pts - Bugfixes)) ``` ## Checklist - [x] Release notes added (see link above) - [x] No obvious regressions in affected areas - [x] Self-review has been performed - I understand what each change in the code does and why it is needed <!--- actual-bot-sections ---> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-14 22:04:43 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#21439