develocity-build-config-samples
develocity-build-config-samples copied to clipboard
Replace `M1` tag in `capture-processor-arch` samples with more generic term
With M2 being made available, we should consider replacing the M1 tag in the capture-processor-arch samples with a more generic term (e.g. Apple Silicon).
See also: https://github.com/slackhq/kotlin-cli-util/blob/main/src/main/kotlin/slack/cli/AppleSiliconCompat.kt
Just a thought: this is why we don't add these things to CCUD. :)
Otherwise if we change this, it would break existing workflows. For example if a team were searching their dashboards for M1 they would suddenly not be able to find this if we changed the name. If we keep these in the samples, then teams can choose how to upgrade or change this.
That is a good point. I am okay with closing this issue.
I think we should update the sample for the very reason that it is a sample that can be manipulated by other developers when copy pasted. 👍 Also people don't tend to update "sample code".