Skip to content

add: Section on fire in CLI scenario#924

Merged
mpoulin merged 1 commit into
realpython:masterfrom
mrshu:mrshu/cli-fire
Oct 10, 2018
Merged

add: Section on fire in CLI scenario#924
mpoulin merged 1 commit into
realpython:masterfrom
mrshu:mrshu/cli-fire

Conversation

@mrshu

@mrshu mrshu commented Oct 6, 2018

Copy link
Copy Markdown
Contributor

Signed-off-by: mr.Shu mr@shu.io

* Add a section on [Python Fire](https://github.com/google/python-fire/)
  in the CLI scenario.

Signed-off-by: mr.Shu <mr@shu.io>
@mpoulin

mpoulin commented Oct 8, 2018

Copy link
Copy Markdown
Contributor

Should we add a link to the announcement on the Google blog ?

@mrshu

mrshu commented Oct 9, 2018

Copy link
Copy Markdown
Contributor Author

@mpoulin I was under the impression that the page should primarily link to "canonical documentation" or "project repository", which is also what the other sections in the docs/scenarios/cli.rst file do. If you think linking to the announcement would be beneficial, I am happy to add it!

@mpoulin

mpoulin commented Oct 10, 2018

Copy link
Copy Markdown
Contributor

The announcement mentions some specific features and benefits we might want to highlight, such as

  • requires no changes to your code
  • is fully automatic (works by inspecting the object at run time)
  • works with any type of object
  • has a built in interpreter mode

@mrshu

mrshu commented Oct 10, 2018

Copy link
Copy Markdown
Contributor Author

Thanks @mpoulin, that makes sense (although I tried my best to highlight that in fire's description).

Would something like "You can learn more about Python Fire in its announcement blogpost" be appropriate here?

I am happy to go with whatever fits python-guide the best in stylistic sense.

@mpoulin

mpoulin commented Oct 10, 2018

Copy link
Copy Markdown
Contributor

Let's see what @dbader thinks.

@dbader

dbader commented Oct 10, 2018

Copy link
Copy Markdown
Member

My 2 cents: I like what's in the current changeset because it's consistent with the rest of the CLI section. The announcement post likely becomes less and less relevant over time whereas the project homepage will always be useful.

So I'm a +1 on merging this as is 🙂

@mpoulin

mpoulin commented Oct 10, 2018

Copy link
Copy Markdown
Contributor

I guess that settles it.
Would you like me to do the merge, Dan?

@dbader

dbader commented Oct 10, 2018 via email

Copy link
Copy Markdown
Member

@mpoulin mpoulin merged commit 4e6093a into realpython:master Oct 10, 2018
@mrshu

mrshu commented Oct 10, 2018

Copy link
Copy Markdown
Contributor Author

Thank you both!

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.

3 participants