Skip to content

feat: Codecleanup for Artifactory uploader#381

Merged
MalavikaKrishnan100 merged 20 commits intofeature/ChocoPackages_Discoveryfrom
feature/Artifactoryuploadercleanup
Feb 4, 2026
Merged

feat: Codecleanup for Artifactory uploader#381
MalavikaKrishnan100 merged 20 commits intofeature/ChocoPackages_Discoveryfrom
feature/Artifactoryuploadercleanup

Conversation

@MalavikaKrishnan100
Copy link
Collaborator

@MalavikaKrishnan100 MalavikaKrishnan100 commented Jan 20, 2026

code cleanup for the artifactory uploader
added the api xml documentation
removed the unwanted lines

…protected/private, sync/async). The summary should briefly describe the purpose inferred from the method name. Include <param> tags for each parameter and <returns> when applicable. For async methods returning Task/Task<T>, use “Asynchronously …”. Example format: /// <summary> /// {Brief purpose} /// </summary> /// <param name="paramName">Description.</param> /// <returns>{Description or "void"}.</returns>

2. Insert regions (create only if missing) with proper #endregion:
• #region Fields
• #region Properties
• #region Constructors
• #region Methods
• #region Events
3. Preserve existing headers, license comments, and using directive order. Skip interface types for regions.
@MalavikaKrishnan100 MalavikaKrishnan100 changed the base branch from main to development January 20, 2026 05:41
@sumanthkb44
Copy link
Collaborator

  • Change the target to feature/ChocoPackages_Discovery
  • Pipelines are not running
  • Add proper description

@sumanthkb44 sumanthkb44 added work in progress started working on the issue and is in progress Code CleanUp labels Jan 20, 2026
@MalavikaKrishnan100 MalavikaKrishnan100 changed the base branch from development to feature/choco_logging_for_dev January 20, 2026 09:05
@MalavikaKrishnan100 MalavikaKrishnan100 changed the base branch from feature/choco_logging_for_dev to feature/ChocoPackages_Discovery January 20, 2026 09:11
@sumanthkb44 sumanthkb44 requested a review from crvreddy January 22, 2026 04:31
@crvreddy
Copy link
Collaborator

image

sonarqube quality checks failed.

@MalavikaKrishnan100
Copy link
Collaborator Author

sonar qube issue is fixed
image

@crvreddy
Copy link
Collaborator

Fix all misspelled warnings in all files

image

@crvreddy
Copy link
Collaborator

Summery not added in below files,check and update
Update proper naming conventions for all files

image

@MalavikaKrishnan100 MalavikaKrishnan100 merged commit 6e1bf51 into feature/ChocoPackages_Discovery Feb 4, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code CleanUp work in progress started working on the issue and is in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants