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

Rename the DWYU output group #170

Merged
merged 1 commit into from
Dec 23, 2023
Merged

Rename the DWYU output group #170

merged 1 commit into from
Dec 23, 2023

Conversation

martis42
Copy link
Owner

There is no strong reason for this change. I am however not happy with the name chosen by me in the early project phase. Since, version 0.1.0 will soonish happen (aka jumping from alpha status to beta level) I want to do this breaking change now, before the project enters a phase in which such changes no longer should happen without strong reasons. Some concrete issues with the old name:

  • The cc_ prefix does not provide any meaningful information
  • The _output postfix is redundant for something used with --output_groups

There is no strong reason for this change. I am however not happy with
the name chosen by me in the early project phase. Since, version 0.1.0
will soonish happen (aka jumping from alpha status to beta level)
I want to do this breaking change now, before the project enters a
phase in which such changes no longer should happen without strong reasons.
Some concrete issues with the old name:
- The `cc_` prefix does not provide any meaningful information
- The `_output` postfix is redundant for something used with `--output_groups`
@martis42 martis42 merged commit b786508 into main Dec 23, 2023
4 checks passed
@martis42 martis42 deleted the rename_output_group branch December 23, 2023 13:53
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.

1 participant