Conversation
brownsarahm
left a comment
brownsarahm
left a comment
There was a problem hiding this comment.
good start, some mostly cosmetic/ simplifyig changes
| or if any error occurs while reading/parsing. | ||
| ''' | ||
| try: | ||
| #r opens the file for reading in text mode, and encoding="utf-8" tells Python to decode the file's bytes |
There was a problem hiding this comment.
| #r opens the file for reading in text mode, and encoding="utf-8" tells Python to decode the file's bytes | |
| # opens the file for reading in text mode, and encoding="utf-8" tells Python to decode the file's bytes |
| def helper(): | ||
|
|
||
| data = read_yaml(MASTER_FILE) | ||
| return data |
There was a problem hiding this comment.
what is the point of this function
| #master dictionary file: to /src/githelp/data/overlays/tips.yml | ||
| MASTER_FILE = os.path.join(DATA_DIR, "tips.yml") |
There was a problem hiding this comment.
let's not use the term mater try mainor even just tips
| #set data to the master dictionary file | ||
| data = helper() | ||
| #check to see if the master is a dict and not empty |
There was a problem hiding this comment.
again, remove references to master and why not jsut use the read function directly?
| if isinstance(data, dict): | ||
| #get the section for this specific command and store it in section | ||
| section = data.get(command) | ||
| #check if section is a dict | ||
| if isinstance(section, dict): |
There was a problem hiding this comment.
if no else, then no need for if
| for key, value in data.items(): | ||
| if isinstance(value, dict): | ||
| names.append(key) |
There was a problem hiding this comment.
why do the values inside need to also be a dict?
| names.append(key) | ||
|
|
||
| names.sort() | ||
| return names |
There was a problem hiding this comment.
wouldn't it be equivalent to just return sorted(data.keys())? since data will always be a dict (even if empty)
| ------- | ||
| str | ||
| A formatted multi-line string | ||
| ''' |
There was a problem hiding this comment.
I think this could be a lot simpler, using fstrings.
| @githelp_cli.command(name="list", help="List of available githelp tip pages.") | ||
| def list_cmd(): | ||
| '''List all available githelp tip.''' | ||
| click.echo(render_menu()) |
There was a problem hiding this comment.
| @githelp_cli.command(name="list", help="List of available githelp tip pages.") | |
| def list_cmd(): | |
| '''List all available githelp tip.''' | |
| click.echo(render_menu()) | |
| @githelp_cli.command( help="List of available githelp tip pages.") | |
| def list(): | |
| '''List all available githelp tip.''' | |
| click.echo(render_menu()) |
this is equivalent but slightly more compact
3 participants
No description provided.