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

Cleanup: move pl.lapp and pl.config into pl.app #417

Open
Tieske opened this issue Feb 13, 2022 · 2 comments
Open

Cleanup: move pl.lapp and pl.config into pl.app #417

Tieske opened this issue Feb 13, 2022 · 2 comments

Comments

@Tieske
Copy link
Member

Tieske commented Feb 13, 2022

pl.app already has a command line parser, pl.lapp is just a more powerful one. But they are both application level utilities, hence should both live in pl.app.

Since pl.config is also an application level tool, it should also go in pl.app.

@alerque
Copy link
Member

alerque commented Feb 15, 2022

Any idea what the history is behind having two app modules and two arg parsers? I don't even think lapp even hit my radar as existing — or I never happened to notice it at the same time as app. Somehow I never realized there were even two implementations.

@Tieske
Copy link
Member Author

Tieske commented Feb 16, 2022

I think the lapp one is more commonly used, but I may be skewed towards that one since we use it at Kong...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants