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.

Sign up for GitHub

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

Jump to bottom

Warn if subcomponent is not added #278

Open
sweh opened this issue Feb 10, 2022 · 1 comment · May be fixed by #420
Open

Warn if subcomponent is not added #278

sweh opened this issue Feb 10, 2022 · 1 comment · May be fixed by #420
Assignees
@elikoga
Milestone
batou 2.5

Comments

@sweh
Copy link
Contributor

sweh commented Feb 10, 2022

If one forgets to add a subcomponent (self +=), no error is show, and of course nothing happens. This may lead to long debugging sessions. batou should warn the user that a configured subcomponent was not added and will do nothing in verify/update cycle.
@zagy
Copy link
Member

zagy commented Feb 10, 2022

E.g.

class Foo(Component):
    def configure(self):
        File("/foo/bar")

@ctheune ctheune added this to the batou 2.4 milestone Aug 24, 2022
@elikoga elikoga self-assigned this Aug 31, 2023
@elikoga elikoga linked a pull request Dec 19, 2023 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@elikoga elikoga

Labels
None yet
Projects
None yet
Milestone
batou 2.5
Development

Successfully merging a pull request may close this issue.

Track component initialisations for error messages flyingcircusio/batou
4 participants
@zagy @ctheune @sweh @elikoga