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

Add new recipe to the C-Cookbook #58

Closed
wants to merge 1 commit into from

Conversation

rongrw
Copy link
Contributor

@rongrw rongrw commented Jan 31, 2017

Updated the C-Cookbook (Cookbook.swim) by adding a new recipe titled "Direct Access to Perl variables". I've placed this recipe at the end of the section "Meat & Potatoes".

Also created a new file called "CONTRIBUTING".

@wbraswell
Copy link
Contributor

@ingydotnet @sisyphus
This pull request affects documentation only, and the contents seems to have been well-thought-out.
I suggest merging.

@ingydotnet
Copy link
Owner

ingydotnet commented Feb 1, 2017

I don't understand. Why are you putting doc/Inline/C/Cookbook.pm into ./Cookbook.swim? The CONTRIBUTING file is a generated file for CPAN. It doesn't belong in the repo.

Please read and comprehend the About file and try again.

@wbraswell
Copy link
Contributor

@ingydotnet
The content was put into a .swim file based on the recommendations of @mohawk2 and @daoswald here:
#56

The CONTRIBUTING file was also a recommendation by @daoswald ; while CONTRIBUTING may be automatically created by dzilla, this is not the case with the EU::MM projects I've worked on, and even GitHub's own docs talk about manually created this file:
https://help.github.com/articles/setting-guidelines-for-repository-contributors/#adding-a-contributing-file

We've all been trying our best to help @rongrw do the right thing, but since it does not meet with your approval, then it seems like you should provide some official guidance for @rongrw by clearly stating where do you want him to put the new content???

@rongrw
Copy link
Contributor Author

rongrw commented Feb 2, 2017

Hi @ingydotnet , @wbraswell ,
Yes, I didn't place the Cookbook.swim into the correct directory. This was certainly not intentional and I do apologise for that. Unfortunately I'm still in the process of learning the github web interface.

I would like to see the new recipe make its way into the C-Cookbook because I find this doc to be such an excellent learning tool with really interesting examples. To that end, if I could be given some further advice to get the changes approved, I'll do that.

Thank you for guiding me thus far, and I do realise that you are all very busy.

@ingydotnet
Copy link
Owner

@rongrw, your contribution looks fine, but why on earth would you copy
./doc/Inline/C/Cookbook.swim to ./Cookbook.swim?

I don't have a CONTRIBUTING file in any of my repos, and I'm not going to add
one here. There's a CONTRIBUTING file in the CPAN dist.

Please change doc/Inline/C/Cookbook.swim in place and get rid of the
CONTRIBUTING file.

@ingydotnet
Copy link
Owner

I fixed/applied your PR myself. Thanks.

@ingydotnet ingydotnet closed this Feb 2, 2017
@perlpunk
Copy link
Collaborator

I'd vote for a CONTRIBUTING file in the repo too, because people are working with the repo and not the CPAN dist, when they want to contribute. Also, github has support for it.

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.

None yet

4 participants