Skip to content
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

feat: added profile-project command #5780

Merged
merged 2 commits into from
Feb 27, 2024
Merged

feat: added profile-project command #5780

merged 2 commits into from
Feb 27, 2024

Conversation

thsig
Copy link
Collaborator

@thsig thsig commented Feb 26, 2024

What this PR does / why we need it:

Added a simple utility command for profiling projects, the profile-project command.

This command logs the number of files included in each action and module in the project (omitting actions that were converted from modules to avoid duplication), their include/exclude configs (if any), and finally the number of modules, actions and tracked files in the project.

This should be useful when helping users with large projects diagnose slow init performance and how to best structure their includes, excludes and .gardenignore.

@thsig thsig requested a review from eysi09 February 26, 2024 11:37
@thsig thsig force-pushed the profile-project-command branch 2 times, most recently from ad13dee to beeedb8 Compare February 26, 2024 11:42
@edvald
Copy link
Collaborator

edvald commented Feb 26, 2024

Could we move this to garden util profile-project? Feels like an odd one to put top level.

@thsig
Copy link
Collaborator Author

thsig commented Feb 26, 2024

Yeah, good point. Will do that.

Added a simple utility command for profiling projects, the
`garden util profile-project` command.

This command logs the number of files included in each action and module
in the project (omitting actions that were converted from modules to
avoid duplication), their include/exclude configs (if any), and finally
the number of modules, actions and tracked files in the project.

This should be useful when helping users with large projects diagnose
slow init performance and how to best structure their includes,
excludes and `.gardenignore`.
@thsig thsig enabled auto-merge February 27, 2024 11:42
@thsig thsig added this pull request to the merge queue Feb 27, 2024
Merged via the queue into main with commit c83f815 Feb 27, 2024
41 checks passed
@thsig thsig deleted the profile-project-command branch February 27, 2024 14:09
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.

2 participants