merge acs-api-commands project into tools - #11098
Conversation
added 4.9 and updated readme
Updated README and added 4.10 commands
4.15 changes
4.16.0 changes
4.17.0 Release diff
4.18 api docs
Add 4.19 and 4.20 api commands
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #11098 +/- ##
=========================================
Coverage 19.65% 19.65%
- Complexity 19794 19798 +4
=========================================
Files 6368 6368
Lines 574889 574913 +24
Branches 70353 70352 -1
=========================================
+ Hits 112985 113009 +24
+ Misses 449634 449633 -1
- Partials 12270 12271 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull Request Overview
This PR merges the existing acs-api-commands project into the tools/apidoc-versions-diff directory (preserving its history), removes the old license file, and updates the release notes accordingly.
- Added a subtree for tools/apidoc-versions-diff with diff.txt and diff.json for versions 4.19→4.20 (and earlier).
- Removed the old license file and updated release notes.
- No functional changes to existing code; purely a build/CI and tooling import.
Reviewed Changes
Copilot reviewed 31 out of 52 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tools/apidoc-versions-diff/diff-419-420/diff.txt | Imported raw API commands diff for CloudStack 4.19→4.20 |
| tools/apidoc-versions-diff/diff-419-420/diff.json | Imported structured JSON diff for CloudStack 4.19→4.20 |
Comments suppressed due to low confidence (2)
tools/apidoc-versions-diff/diff-419-420/diff.txt:17
- [nitpick] Command descriptions should start with an uppercase verb in third-person singular (e.g.,
Deletes a backup repository).
deleteBackupRepository (delete a backup repository)
tools/apidoc-versions-diff/diff-419-420/diff.txt:29
- The command name uses
Nsxrather thanNSX; consider renaming todeleteNSXControllerto reflect the acronym consistently.
deleteNsxController (delete NSX Controller to CloudStack)
|
|
||
| purgeExpungedResources (Purge expunged resources) | ||
|
|
||
| deleteNsxController (delete NSX Controller to CloudStack) |
There was a problem hiding this comment.
The description reads 'to CloudStack' but should say 'from CloudStack' to accurately reflect removal.
| deleteNsxController (delete NSX Controller to CloudStack) | |
| deleteNsxController (delete NSX Controller from CloudStack) |
|
@pdion891 , are you still going through with this? |
|
Hi @pdion891 is this PR still in progress? |
|
Hi @nvazquez , I have not kept track of this PR recently. I will rebase the PR branch... |
- .codespellrc stops the spelling check from failing on the generated files' typos.
|
recent changes to get the ci-cd pipeline to work:
|
Description
As suggested in the mailing list, here is the merge of acs-api-commands/
which is use to create CloudStack Release Notes to extract new or update api's.
Project renamed into folder
tools/apidoc-versions-diffonce this PR merge, the project acs-api-commands/ will be mark as archived/deprecated with a link to this folder.
Perform using git subtree which kept previous commit histories;
Types of changes
Feature/Enhancement Scale
Bug Severity
How Has This Been Tested?
No.
How did you try to break this feature and the system with this change?
Did not.