Skip to content

Remove arch config option from Seqera MachineRequirement#7063

Merged
pditommaso merged 2 commits intomasterfrom
remove-seqera-machinereq-arch
Apr 22, 2026
Merged

Remove arch config option from Seqera MachineRequirement#7063
pditommaso merged 2 commits intomasterfrom
remove-seqera-machinereq-arch

Conversation

@pditommaso
Copy link
Copy Markdown
Member

Summary

  • Remove the redundant arch option under seqera.executor.machineRequirement.
  • Architecture is now sourced exclusively from the process arch directive via task.getContainerPlatform() (already passed as taskArch to SchemaMapperUtil.toMachineRequirement).
  • Update related unit tests.

Test plan

  • ./gradlew :plugins:nf-seqera:test passes

🤖 Generated with Claude Code

The `arch` config option under `seqera.executor.machineRequirement`
is redundant: the architecture is already derived from the process
`arch` directive via `task.getContainerPlatform()`. Drop the config
option and source arch exclusively from the task definition.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 22, 2026

Deploy Preview for nextflow-docs-staging ready!

Name Link
🔨 Latest commit 17bd338
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/69e8ec701069a30008465978
😎 Deploy Preview https://deploy-preview-7063--nextflow-docs-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pditommaso pditommaso requested a review from bentsherman April 22, 2026 14:38
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
@pditommaso pditommaso requested a review from a team as a code owner April 22, 2026 15:42
@pditommaso pditommaso merged commit da06e9a into master Apr 22, 2026
24 checks passed
@pditommaso pditommaso deleted the remove-seqera-machinereq-arch branch April 22, 2026 15:54
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.

2 participants