nextflow icon indicating copy to clipboard operation
nextflow copied to clipboard

Fix stack overflow error with workflow metadata

Open bentsherman opened this issue 7 months ago • 2 comments

Close #5986

To be fair, I'm not entirely sure why this getter was here in the first place. Looking at this commit it seems to be related to the execution report. I guess we'll see what the tests have to say...

bentsherman avatar Apr 19 '25 00:04 bentsherman

Deploy Preview for nextflow-docs-staging ready!

Name Link
Latest commit adc0057ae21370db791ab79fa76086bfabc0d857
Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/685f31c8d7e9590008c11f29
Deploy Preview https://deploy-preview-5988--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.

netlify[bot] avatar Apr 19 '25 00:04 netlify[bot]

Yeah, it seems has been added to make it work the workflow report. better verify there's. no regression with it

pditommaso avatar Apr 22 '25 10:04 pditommaso

Any chance to capture in a test what's fixing this?

pditommaso avatar Jun 18 '25 12:06 pditommaso

Added a unit test, also checked the report and it is still rendering correctly

bentsherman avatar Jun 18 '25 13:06 bentsherman

Nice

pditommaso avatar Jun 18 '25 14:06 pditommaso