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".