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_lang: nested meta sections #51

Open
oliver-sanders opened this issue Aug 4, 2021 · 0 comments
Open

cylc_lang: nested meta sections #51

oliver-sanders opened this issue Aug 4, 2021 · 0 comments
Labels
bug Something isn't working
Milestone

Comments

@oliver-sanders
Copy link
Member

Low priority bug (more a limitation than a bug) easier just to document for now in the (hopefully) unlikely case that we will need this (famous last words I will likely eat if we start giving Cylc plugins configuration sections...).

As of #50 only settings explicitly defined in meta sections are documented.

Example:

[platforms]
    [[<platform name>]]
        host
       [[[selection]]]
           method

    [[localhost]]
        host = localhost

The above example will document fine, the overwritten host setting will appear correctly.

However, if we wanted to overwrite the [selection]method for localhost that would not appear in the docs.

I think fixing this would require making cylc.flow.parsec.config.ConfigNode.walk aware of the meta keyword.

See #50

@oliver-sanders oliver-sanders added the bug Something isn't working label Aug 4, 2021
@oliver-sanders oliver-sanders added this to the pending milestone Aug 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant