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

Cylc lint: U999 is over-eager #6685

Open
ColemanTom opened this issue Mar 26, 2025 · 0 comments
Open

Cylc lint: U999 is over-eager #6685

ColemanTom opened this issue Mar 26, 2025 · 0 comments
Labels
bug Something is wrong :(

Comments

@ColemanTom
Copy link
Contributor

I found U999 was complaining about something it should not be complaining about. Take this snippet.

$ cat flow.cylc
    [[xtriggers]
        foo = abc(a=123, host=hello)
        foo = abc(a=123, \
            host=hello \
        )
$ CYLC_VERSION=8.4 cylc lint $PWD
[U999] flow.cylc:4: [runtime][__MANY__][workflow state polling]host - not available at Cylc 8

Checked /g/ns/sc/user-cylc/tcoleman/code/basic-workflow against ['728', 'style'] rules and found 1 issue.
$ CYLC_VERSION=8.4 cylc version
8.4.1

When host is on a new line, it throws an error, thinking it belongs to workflow state polling, even though it is part of an xtrigger and host is a completely valid and reasonable kwarg.

@ColemanTom ColemanTom added the bug Something is wrong :( label Mar 26, 2025
@ColemanTom ColemanTom changed the title Cylc lint: U9999 is over-eagre Cylc lint: U999 is over-eagre Mar 26, 2025
@MetRonnie MetRonnie changed the title Cylc lint: U999 is over-eagre Cylc lint: U999 is over-eager Mar 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is wrong :(
Projects
None yet
Development

No branches or pull requests

1 participant