Skip to content

Rename terraform modules packages.rb file to namespace_packages.rb

Moaz Khalifa requested to merge 437927-Rename-terraform-modules-packages.rb-file-to-namespace_packages.rb into master

Context

In New API class for terraform modules project end... (!137824 - merged), we introduced a new project-level API class, and the purpose of this class was to include all project-level endpoints in the Terraform Registry. Thus, we moved all project-level endpoints from the existing API::Terraform::Modules::V1::Packages class to the new API::Terraform::Modules::V1::ProjectPackages class.

Now all endpoints in the API::Terraform::Modules::V1::Packages class are namespace-level ones. Therefore, this class should be renamed to API::Terraform::Modules::V1::NamespacePackages to reflect its purpose accurately and to be consistent with the naming convention we have for the class that contains the project-level endpoints and the one contains the namespace-level ones.

What does this MR do and why?

  • Rename API::Terraform::Modules::V1::Packages class to API::Terraform::Modules::V1::NamespacePackages.
  • Rename the specs file.
  • Remove rubocop exceptions and fix violations.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Related to #437927 (closed)

Edited by Moaz Khalifa

Merge request reports