+
Skip to content

Conversation

jfy133
Copy link
Member

@jfy133 jfy133 commented Sep 1, 2025

To close #3740

PR checklist

  • This comment contains a description of changes (with reason)
  • CHANGELOG.md is updated
  • If you've fixed a bug or added code that should be tested, add tests!
  • Documentation in docs is updated

@jfy133
Copy link
Member Author

jfy133 commented Sep 1, 2025

My only fear is that if the pipeline description is too long we would lose the hashtags and URL... I considered adding teh description underneath all of them, but it looked ugly... so IDK the better method

Copy link
Contributor

@mashehu mashehu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would say let's see if it actually a problem, there seems to be a character limit of 350 for the description field, so should have enough space for the hashtags

Copy link
Member

@mirpedrol mirpedrol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jfy133
Copy link
Member Author

jfy133 commented Oct 1, 2025

Any chance either of you can resolve the conflict @mashehu @mirpedrol ? Won't be at my computer for a bit and I may forget 😅

@mashehu mashehu enabled auto-merge October 1, 2025 10:42
@mashehu mashehu merged commit 2e6407b into nf-core:dev Oct 1, 2025
115 checks passed
@jfy133
Copy link
Member Author

jfy133 commented Oct 1, 2025

Thank youuuuu

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.

4 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载