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 perldata.pod else everybody will think sigil=$ #22429

Open
wants to merge 1 commit into
base: blead
Choose a base branch
from

Conversation

jidanni
Copy link
Member

@jidanni jidanni commented Jul 26, 2024

No matter all my life I thought "Sigil" only meant "$".

It's not because "S" looks like "$", it was because of this one sided example!

Well, no more discrimination allowed in this sentence. Equal opportunity for all!

No matter all my life I thought "Sigil" only meant "$".

It's not because "S" looks like "$", it was because of this one sided example! 

Well, no more discrimination allowed in this sentence. Equal opportunity for all!
@rwp0
Copy link
Contributor

rwp0 commented Jul 26, 2024

Haha, cool!

@guest20
Copy link

guest20 commented Jul 26, 2024

The section of the document you're patching is about special variables that don't obey normal parsing rules. I don't think @1 is a useful example (since it's in *1 as reserved by regex match vars)

You might be better served to add a "sigils" heading above the paragraphs that explain each sigil, then add a heading for each sigil above each paragraph...

@jidanni
Copy link
Member Author

jidanni commented Jul 27, 2024

Personally I also didn't want to "send the other two children home" in order to "replace them with members of underrepresented groups."

So... please anybody fix this page one way or another. Any way is fine with me. As far as me editing my edit... that would make a Git Mess surely. So please just change the page as you see fit. Thanks.

@guest20
Copy link

guest20 commented Jul 28, 2024

What's all this about children?

@jidanni
Copy link
Member Author

jidanni commented Jul 29, 2024

There were three items. I didn't want to add more items, so I changed two of the three.

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