Skip to content

Listing of cookbooks and recipes from chef repositories - #628

Open
nikhil2611 wants to merge 2 commits into
mainfrom
nikhil/INFWS-139-list-cookbooks-from-repo
Open

Listing of cookbooks and recipes from chef repositories #628
nikhil2611 wants to merge 2 commits into
mainfrom
nikhil/INFWS-139-list-cookbooks-from-repo

Conversation

@nikhil2611

@nikhil2611 nikhil2611 commented Jul 26, 2022

Copy link
Copy Markdown
Contributor

Signed-off-by: nikhil2611 nikhilgupta2102@gmail.com

Description

Listing the chef repo cookbooks and recipes when the user added the repositories

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (non-breaking change that does not add functionality or fix an issue)

Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
nikhil2611 requested review from a team as code owners July 26, 2022 05:19
@vkarve-chef

Copy link
Copy Markdown
Collaborator

@nikhil2611 this change results in all folder content getting enumerated in the tree view. Can we filter it so only cookbooks folder and then only recipes folder within it is enumerated?

Screenshot 2022-08-01 at 12 16 32 PM

Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@sonarqubecloud

sonarqubecloud Bot commented Aug 1, 2022

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 1 Security Hotspot
Code Smell A 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@nikhil2611

Copy link
Copy Markdown
Contributor Author

@nikhil2611 this change results in all folder content getting enumerated in the tree view. Can we filter it so only cookbooks folder and then only recipes folder within it is enumerated?

Screenshot 2022-08-01 at 12 16 32 PM

I have filtered out most of the files and folders not needed , just need to add that element name in the array.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants