Skip to content

feat: Code clean up in package identifier and package creator#384

Merged
Nivetha208 merged 3 commits intofeature/ChocoPackages_Discoveryfrom
feature/Codecleanup_PacIdendcrea
Jan 30, 2026
Merged

feat: Code clean up in package identifier and package creator#384
Nivetha208 merged 3 commits intofeature/ChocoPackages_Discoveryfrom
feature/Codecleanup_PacIdendcrea

Conversation

@Nivetha208
Copy link
Collaborator

Code cleanup done in Package Identifier and Package Creator project like Adding summary above each methods, Removing unused imports, segregating properties, fields, constructors and methods and updating proper naming conventions.

Sridhar, NIVETHA (ext) (FT D AA IN SGI EA EBT BE) and others added 2 commits January 27, 2026 12:49
@crvreddy
Copy link
Collaborator

Fix Naming warnings in all files.

crvreddy
crvreddy previously approved these changes Jan 27, 2026
Copy link
Collaborator

@crvreddy crvreddy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding summery for methods missed for Nugetprocessor.cs file

@crvreddy crvreddy dismissed their stale review January 27, 2026 10:20

new issue identified

Copy link
Collaborator

@crvreddy crvreddy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@sumanthkb44
Copy link
Collaborator

@Nivetha208 feature/Codecleanup_PacIdendcrea let the branch name be self explementary , instead of in coded format.

@sumanthkb44
Copy link
Collaborator

Will also add SonarQube status as screenshot here

@Nivetha208
Copy link
Collaborator Author

Will also add SonarQube status as screenshot here

image

@Nivetha208 Nivetha208 merged commit 1756406 into feature/ChocoPackages_Discovery Jan 30, 2026
10 checks passed
@crvreddy crvreddy deleted the feature/Codecleanup_PacIdendcrea branch February 10, 2026 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants