Repository navigation
Version update - #276
Version update#276
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR finalizes the version metadata for the XOOPS 2.7.4 release by replacing the RC2 identifier with the stable version and updating the corresponding changelog. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing: 2
✨ Finishing Touches
Comment |
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="htdocs/include/version.php" line_range="35" />
<code_context>
* Define XOOPS version
*/
-define('XOOPS_VERSION', 'XOOPS 2.7.4-RC2');
+define('XOOPS_VERSION', 'XOOPS 2.7.4');
</code_context>
<issue_to_address>
**Final release has prerelease guidance**
When the final 2.7.4 build is published with the unchanged README, `XOOPS_VERSION` identifies the build as stable, but `docs/README.txt` still calls it RC 2, asks users to test before the final release, points to RC 2 downloads, and includes RC1 upgrade guidance, so users receive contradictory release information and instructions.
Update `docs/README.txt` to identify the final 2.7.4 release and align its announcement, download label, testing instructions, and upgrade guidance with that status.
</issue_to_address>Sourcery assessment
Approval pending. 1 finding to address first.
Blocking findings: htdocs/include/version.php:35
| * Define XOOPS version | ||
| */ | ||
| define('XOOPS_VERSION', 'XOOPS 2.7.4-RC2'); | ||
| define('XOOPS_VERSION', 'XOOPS 2.7.4'); |
There was a problem hiding this comment.
🟡 Medium · Final release has prerelease guidance
When the final 2.7.4 build is published with the unchanged README, XOOPS_VERSION identifies the build as stable, but docs/README.txt still calls it RC 2, asks users to test before the final release, points to RC 2 downloads, and includes RC1 upgrade guidance, so users receive contradictory release information and instructions.
Update docs/README.txt to identify the final 2.7.4 release and align its announcement, download label, testing instructions, and upgrade guidance with that status.
Prompt for AI agents
In `htdocs/include/version.php` at line 35:
**Final release has prerelease guidance**
When the final 2.7.4 build is published with the unchanged README, `XOOPS_VERSION` identifies the build as stable, but `docs/README.txt` still calls it RC 2, asks users to test before the final release, points to RC 2 downloads, and includes RC1 upgrade guidance, so users receive contradictory release information and instructions.
Update `docs/README.txt` to identify the final 2.7.4 release and align its announcement, download label, testing instructions, and upgrade guidance with that status.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #276 +/- ##
=========================================
Coverage 22.54% 22.54%
Complexity 8671 8671
=========================================
Files 685 685
Lines 46328 46328
=========================================
Hits 10444 10444
Misses 35884 35884 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Summary by Sourcery
Finalize the XOOPS 2.7.4 release version.
Documentation:
Chores: