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

[tiny] README - rephrase warning for clarity #190

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[tiny] README - rephrase warning for clarity #190

wants to merge 2 commits into from

Conversation

floer32
Copy link

@floer32 floer32 commented Feb 27, 2021

Please forgive me for over-explaining a tiny/trivial change πŸ˜…

On first glance it looks like this is still vulnerable β€” or so I thought. I think this slight rephrase makes it more clear on first glance.

Given this is a development tool, I think catering to "at first glance" makes sense, since in development we're thinking "just go go go!"

(Versus in production mindset when we make more measured choices; for me that means reading a README more slowly for production, versus jumping around more in dev mindset.)

On first glance it looks like this is still vulnerable β€” or so I thought. I think this slight rephrase makes it more clear on first glance. Given this is a development tool, I think catering to "at first glance" makes sense, since in development we're thinking "just go go go!" (Vs in production where we make more measured choices; for me that means reading a README more slowly for production)
fix a typo
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