Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More consistent code conventions for e.g. indentation #517

Open
vdauwera opened this issue Feb 19, 2025 · 1 comment
Open

More consistent code conventions for e.g. indentation #517

vdauwera opened this issue Feb 19, 2025 · 1 comment
Labels
improve New feature or request question Further information is requested

Comments

@vdauwera
Copy link
Collaborator

As noted previously by @mribeirodantas:

          I wonder if we shouldn't stick to nf-core conventions on code indentation, i.e.
process FOO {
    publishDir ...
    some other directive ...

    input:
    val foo
    path bar

    output:
    stdout

    script:
    """

instead of

 process FOO {

    publishDir ...
    some other directive ...

    input:
        val foo
        path bar

    output:
        stdout

    script:
    """

It's not clear to me if the way we're doing it contributes a lot to readability/understanding and if it's worth teaching people differently from how most of the community writes Nextflow pipelines.

Originally posted by @mribeirodantas in #431 (comment)

@vdauwera vdauwera added improve New feature or request question Further information is requested labels Feb 19, 2025
@vdauwera
Copy link
Collaborator Author

As noted in #439 this dovetails with the need to align with the vscode linting.

@vdauwera vdauwera changed the title More consistent conventions on code indentation More consistent code conventions for e.g. indentation Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improve New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant