-
Notifications
You must be signed in to change notification settings - Fork 9
Release r3.2 (Fall'25 M4) #103
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release r3.2 (Fall'25 M4) #103
Conversation
|
@albertoramosmonagas here is the list of changes since r3.1 if you would create a draft release:
Please select the relevant ones and combine with the change(s) within r3.1. And please merge |
Thanks for the summary. I’ve already included all the changes listed since r3.1 in the draft changelog for r3.2. Just one quick clarification: when you mention “combine with the change(s) within r3.1”, I assumed those remain part of the previous r3.1 release and didn’t carry them over into r3.2. Would you expect the r3.1 changes to be repeated in the r3.2 changelog as well? I’m happy to adjust if needed, just want to make sure we’re on the same page. Regarding main, this release branch was originally created from it and is already fully up to date. Not sure if there’s anything else I should do on my side in that regard — happy to double-check if needed. |
From the template for CHANGELOG.md:
In your case: the section for r3.2 should contain all changes since the last public release (r2.2). In the "Full Changelog" line this is already correct (...compare/r2.2...r3.2)
No, the release branch is 13 commits behind camaraproject:main (reason is that also the main branch in your fork is behind the upstream repository, you would have needed to sync it before creating the branch):
Just click on the these "13 commits behind" and you will get offered to create a pull request from camaraproject:main into your PR branch. This PR you can merge directly yourself (as the PR branch is not protected) and you are done. |
|
Hi @hdamker, Thanks a lot for the explanation. From VSCode everything appeared to be fully up to date, so I hadn’t noticed anything pending. But after checking directly on the fork, I did see it was indeed behind main as you pointed out. I hadn’t received any notification or prompt about that. I’ve now synced it and I believe everything should be in order. Let me know if there’s anything else you’d like me to double-check. |
|
@albertoramosmonagas Thanks for the updates. I suppose that the PR is now "ready for review" and will change the status accordingly. |
Add "public" tag to the release Co-authored-by: Kevin Smith <Kevsy@users.noreply.github.com>
documentation/API_documentation/population-density-data-API-Readiness-Checklist.md
Outdated
Show resolved
Hide resolved
documentation/API_documentation/population-density-data-API-Readiness-Checklist.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Kevin Smith <Kevsy@users.noreply.github.com>
Kevsy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As well as the release-management review comments above, two additional changes to make in population-density-data.yaml:
L5: In the info.description please start the first sentence with 'The' to be consistent with the marketing description on the wiki (i.e. 'The Population Density API')
L317: you have correctly referenced the RFC for the date-time format, but you need to delete the additional text
Recommended format is yyyy-MM-dd'T'HH:mm:ss.SSSZ
(i.e. which allows 2023-07-03T14:27:08.312+02:00 or 2023-07-03T12:27:08.312Z)
The minimum startTime must cover at least 3 months before the request time."
...as per camaraproject/Commonalities#497
Hi, Thanks for the comments. The issue at L5 is now fixed — the description starts with “The Population Density API” to align with the marketing description on the wiki. For L317, I suggest another solution would look like this
This removes the additional text about the recommended format, as per camaraproject/Commonalities#497. Could you please confirm this approach is fine? If so, I will apply it consistently across the rest of the spec |
|
Hi @albertoramosmonagas ,
That PR also removes the text Could this be covered in the |
Changes applied. |
jgarciahospital
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Include minor version Co-authored-by: Kevin Smith <Kevsy@users.noreply.github.com>
d0f4ab3
|
@camaraproject/population-density-data_codeowners |
Kevsy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved on behalf of Release Management 👏
Next steps for the team:
- PR merged (by API repository codeowner)
- Release created within GitHub (by API repository codeowner)
- Release Tracker updated (with creation date of the release and the release tag link)

What type of PR is this?
Add one of the following kinds:
What this PR does / why we need it:
Publication of Fall'25 M4 public release of population-density-data v0.3.0
Which issue(s) this PR fixes:
Fixes #102
Special notes for reviewers:
None
Changelog input
Additional documentation
None