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.

Sign up for GitHub

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

Jump to bottom

Change extensions to .p6 #57

Closed
JJ opened this issue Feb 27, 2019 · 12 comments
Closed

Change extensions to .p6 #57

JJ opened this issue Feb 27, 2019 · 12 comments

Comments

@JJ
Copy link
Contributor

JJ commented Feb 27, 2019

It's the preferred extension today.
@manwar
Copy link
Collaborator

manwar commented Feb 27, 2019

Do you mean every examples in the folder below should have the extension .p6?

https://github.com/perl6/perl6-examples/tree/master/categories/cookbook

@JJ
Copy link
Contributor Author

JJ commented Feb 27, 2019

Right.

@manwar
Copy link
Collaborator

manwar commented Feb 27, 2019

Please expect the next PR with the extension changed to .p6 tomorrow ;-)

@jonathanstowe
Copy link

It's a personal gripe but why do scripts have "extensions" at all? for all OS but Windows they aren't needed, and for Windows they probably need a .BAT wrapper anyway.

@AlexDaniel AlexDaniel mentioned this issue Feb 27, 2019
Open
manwar added a commit to manwar/perl6-examples that referenced this issue Feb 28, 2019
@manwar
Fixed issue Raku#57 by renaming extension .pl to .p6.
bd6fd93
@manwar manwar mentioned this issue Feb 28, 2019
Merged
JJ added a commit that referenced this issue Feb 28, 2019
@JJ
Merge pull request #59 from manwar/fix-issue-57-58
b8e411c 
Fix issue #57 and #58 Thanks!
These tests take too much time.
JJ pushed a commit to JJ/perl6-examples that referenced this issue Jan 4, 2020
@manwar @JJ
Fixed issue Raku#57 by renaming extension .pl to .p6.
0e48622
@JJ
Copy link
Contributor Author

JJ commented Jan 4, 2020

Most examples still have the .pl extension, I'm afraid.

@Altai-man Altai-man self-assigned this Jan 4, 2020
@camstuart
Copy link
Contributor

Should the shebangs now be #!/usr/bin/env raku and extensions .raku?

Happy to provide the PR

@JJ
Copy link
Contributor Author

JJ commented Aug 27, 2020 via email

@Altai-man
Copy link
Member

The purpose of this repo is not very clear, hence there are no volunteers available and it was abandoned for quite some time.
If you want to work on it, firstly a clear image of how exactly this repo will be helpful, is needed.

@JJ
Copy link
Contributor Author

JJ commented Aug 27, 2020 via email

@camstuart
Copy link
Contributor

Thanks @JJ I will do the PR to make this update. I agree there should be tests where possible.
One liners on the other hand might get conflated, as they would need to be factored into sub routines, which might confuse the reader.

I agree also with @Altai-man But I think a new issue is the place to discuss, which I will also set up. To his point, I only found this repo accidental and initially thought it was something else.

@JJ
Copy link
Contributor Author

JJ commented Aug 28, 2020 via email

camstuart added a commit to camstuart/examples that referenced this issue Aug 29, 2020
@camstuart
Update cookbook file extensions and shebangs
e8e44da 
- Changed file extension from .p6 to .raku
- Changed shebangs to #!/usr/bin/env raku
- Set file modes to executable

See also: Raku#57, Raku#58
@camstuart camstuart mentioned this issue Aug 29, 2020
Open
@Altai-man Altai-man removed their assignment Oct 6, 2021
@coke
Copy link
Contributor

coke commented Oct 15, 2021

Closing this issue as it's outdated; the correct extension going forward is .raku. Opening a new ticket to track

@coke coke closed this as completed Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@JJ @coke @manwar @jonathanstowe @camstuart @Altai-man