Skip to content

Fix terms, typos and grammar mistakes in the API, error messages, events, etc. #7857

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vladimirpetrov
Copy link
Contributor

Description

This PR aligns the use of terminology, renaming VM / virtual machine references to 'Instance' and also capitalising the terms Templates, Network, Snapshot, User, Account in CloudStack APIs, error and log messages, events, tooltips, etc. Many typos, grammar and spelling mistakes were fixed, also terms like IPv4, VPN, VPC, etc. were properly capitalised. Some error messages were cleaned for better readability. The test cases, expecting some exception strings were adjusted accordingly.

Types of changes

  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

None

How Has This Been Tested?

This change has been tested locally using the simulator hypervisor plugin.

@codecov
Copy link

codecov bot commented Aug 11, 2023

Codecov Report

Merging #7857 (babc9a2) into main (540dcc1) will not change coverage.
Report is 4 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #7857   +/-   ##
=========================================
  Coverage     14.38%   14.38%           
  Complexity    10072    10072           
=========================================
  Files          2747     2747           
  Lines        258922   258922           
  Branches      40318    40318           
=========================================
  Hits          37241    37241           
  Misses       216870   216870           
  Partials       4811     4811           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

@DaanHoogland DaanHoogland added this to the 4.20.0.0 milestone Sep 29, 2023
@JoaoJandre JoaoJandre modified the milestones: 4.20.0.0, 4.21.0.0 Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants