Skip to content

Use admonition for documenting a tip#6222

Merged
squidfunk merged 1 commit intosquidfunk:masterfrom
sisp:docs/tip-admonition
Oct 30, 2023
Merged

Use admonition for documenting a tip#6222
squidfunk merged 1 commit intosquidfunk:masterfrom
sisp:docs/tip-admonition

Conversation

@sisp
Copy link
Copy Markdown
Contributor

@sisp sisp commented Oct 21, 2023

I've moved a plain tip regarding Python intro into a tip admonition which is more visually pleasing IMO and more consistent with the rest of the fantastic docs.

@squidfunk
Copy link
Copy Markdown
Owner

I completely forgot this PR! It somehow got deleted from my inbox, which is why I'm late to the party. LGTM! Thanks!

@squidfunk squidfunk merged commit 551d98e into squidfunk:master Oct 30, 2023
@sisp sisp deleted the docs/tip-admonition branch October 31, 2023 05:39
@sisp
Copy link
Copy Markdown
Contributor Author

sisp commented Oct 31, 2023

No worries, it happens. 😉 Thanks for liking and merging it. 🙏

@squidfunk
Copy link
Copy Markdown
Owner

Perfect. If you or somebody else has a better primer on Python, happy to replace it. realpython.com tries to force oyu into their paywalled content and/or forces you to sign up after reading 1-2 articles.

@alexvoss
Copy link
Copy Markdown
Collaborator

I was hoping that Code Academy would provide but all I could find was this and it seems a bit thin. The Python documentation has a useful combo of describing pip and venv. Regarding the RealPython tutorial, I think it tries to do too much and does not distinguish between different use cases. Just my 2p.

@squidfunk
Copy link
Copy Markdown
Owner

Hmm, maybe a good idea for a blog article on our own blog then!

@alexvoss
Copy link
Copy Markdown
Collaborator

You read my mind but I am slowly learning to prioritize - mainly by sticking things into a backlog, which I will do with this discussion. Might become a side-product of thinking about making producing tutorial more efficient by reusing content.

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