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

Update README wording #105

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

Conversation

cmdcolin
Copy link

Hi there
This is sort of a silly contribution, but the wording in the README at first glance might make it look like running these eval expressions is somehow "dangerous" e.g. in a js eval type way, but I think this is not the intent

I replaced it with a simple Hello world here just to be a bit more...boring :)

@TomFrost
Copy link
Owner

Hahaha -- Dangerzone is just another element of the Archer theme that the Readme is written with. I see your point though -- the uninitiated may not take it that way! I wonder if there's an in-theme example that makes more sense...

@cmdcolin
Copy link
Author

Lol...the archer thing I always see is the format "Do you want X? Because that's how you get X"

So...

updated PR :)

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.

2 participants