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

Deprecate --profile-type flag #147

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

rsevilla87
Copy link
Member

@rsevilla87 rsevilla87 commented Nov 13, 2024

Type of change

  • Optimization
  • Documentation Update

Description

The flag --profile-type can be deprecated thanks to the new flag --metrics-profile that allows a list of files.

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

@rsevilla87 rsevilla87 added enhancement New feature or request ok-to-test labels Nov 13, 2024
@rsevilla87 rsevilla87 requested review from a team as code owners November 13, 2024 00:03
@rsevilla87 rsevilla87 force-pushed the deprecate-profile-type branch from 083bf77 to d4f099a Compare November 13, 2024 22:16
Signed-off-by: Raul Sevilla <[email protected]>

Update tests

Signed-off-by: Raul Sevilla <[email protected]>
Signed-off-by: Raul Sevilla <[email protected]>
Signed-off-by: Raul Sevilla <[email protected]>
Signed-off-by: Raul Sevilla <[email protected]>
Signed-off-by: Raul Sevilla <[email protected]>
@rsevilla87 rsevilla87 force-pushed the deprecate-profile-type branch from 5df4022 to 40b15af Compare November 14, 2024 20:46
@vishnuchalla
Copy link
Contributor

Seems to be a breaking change. Needs to go in a dedicated release under detailed supervision. Can we have all the repos/CI systems impacting by this change first to start with?

@rsevilla87
Copy link
Member Author

e have all the repos/

yes, it's a breaking change, we can work on it in another release, there's no rush to have it soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ok-to-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants