Skip to content

support reading Argparse instances inside classes#60

Closed
laf0rge wants to merge 1 commit into
sphinx-doc:mainfrom
osmocom:inside-classes
Closed

support reading Argparse instances inside classes#60
laf0rge wants to merge 1 commit into
sphinx-doc:mainfrom
osmocom:inside-classes

Conversation

@laf0rge

@laf0rge laf0rge commented Jul 31, 2024

Copy link
Copy Markdown

So far, shphinxarg only has support for reading/importing Argparse instances from global variables/attributes within a module. However, there are use cases where Argparse is used inside classes of a module, not as a global variable. This is particularly the case for uses of 'argparse' in the context of CLI / REPL style user interfaces, such as for example those built using the 'cmd2' module.

This change introduces the ability to specify a path in ':func:' using the '.'-notation (e.g. 'PysimApp.bulk_script_parser').

Initial patch by: Harald Welte laforge@osmocom.org
Co-authored by: Vadim Yanitskiy fixeria@osmocom.org

This is a follow-up to #36 (comment)

So far, shphinxarg only has support for reading/importing Argparse
instances from global variables/attributes within a module.  However,
there are use cases where Argparse is used inside classes of a module,
not as a global variable.  This is particularly the case for uses
of 'argparse' in the context of CLI / REPL style user interfaces,
such as for example those built using the 'cmd2' module.

This change introduces the ability to specify :func: wit '.' notation
like foo.bar.baz
@codecov

codecov Bot commented Jul 31, 2024

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@laf0rge

laf0rge commented Apr 13, 2026

Copy link
Copy Markdown
Author

superseded by #93

@laf0rge laf0rge closed this Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant