Fix BIOCONDUCTOR_NAME reflecting devel on all release images

[Copilot]

BIOCONDUCTOR_NAME was hardcoded to bioconductor_docker_devel-<arch> in the Dockerfile, so every image — including release tags like RELEASE_3_23 — reported the wrong name at runtime.

Change

Replace the hardcoded devel string with the BIOCONDUCTOR_VERSION ARG (already declared in the same final stage):

# Before
ENV BIOCONDUCTOR_NAME="bioconductor_docker_devel-$TARGETARCH"

# After
ENV BIOCONDUCTOR_NAME="bioconductor_docker_${BIOCONDUCTOR_VERSION}-${TARGETARCH}"

This produces the correct value for every branch at build time without any workflow changes:

Branch	BIOCONDUCTOR_NAME
devel	bioconductor_docker_3.24-amd64
RELEASE_3_23	bioconductor_docker_3.23-amd64

[almahmoud]

I don't think devel should be removed. It was explicitly added to have separate binary directories (since they inherit the name but then also adds version) for the same Bioc version when it's in devel vs release, and the targetarch to open up possibility for arm builds of the binaries. If you want the name to also include version that's fine, we should then maybe change the expected binary path to expect version built into name as opposed to appending it, or just leave as is for the name to differentiate release vs devel and arm vs amd, while the version is a separate env. (But yes when creating the release branch, the name should also be changed to remove devel)