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

Added a new Book and Turorial #924

Merged
merged 2 commits into from
Feb 17, 2017
Merged

Added a new Book and Turorial #924

merged 2 commits into from
Feb 17, 2017

Conversation

packtpartner
Copy link
Contributor

Added a new book (Git: Mastering Version Control) and tutorial (Git: Version Control Made Easy)

Added a new book (Git: Mastering Version Control) and tutorial (Git: Version Control Made Easy)
@peff
Copy link
Member

peff commented Jan 21, 2017

I think the book link is OK (though it would probably look better if moved to the left-hand column, since that is the shortest one right now).

I'm not sure about the tutorial, though. The rest of the tutorial section are all short-ish documents that are free online. Mixing in a longer, paid training course feels out of place. I'm undecided on whether it should go in a separate section for paid training, or if it should be omitted entirely. Obviously we link to for-profit books already, but I'm not sure to what degree we want to get into advertising other resources.

@jnareb
Copy link
Member

jnareb commented Jan 21, 2017

Actually "Git: Mastering Version Control" is a book bundle, composed of three books:

  • Git Essentials, Ferdinando Santacroce
  • Git Version Control Cookbook, Aske Olsson and Rasmus Voss
  • Mastering Git, Jakub Narębski

Though it is presented similar to a book... Just FYI.

@packtpartner
Copy link
Contributor Author

Would it help if I create a new section for paid courses? I think it could prove to be a good resource to help many in their learning of Git.

@stefanbeller
Copy link
Contributor

Would it help if I create a new section for paid courses?

I dunno. On the one hand it is better than mixing it into the section of un-paid courses, but on the other hand we'd need to discuss what to add (or remove) to such a list. Specially when money is involved, I think the Git community wants to not endorse one book and not endorse another one, but rather keep it fair. And from that POV it is easiest to just say no-no to any paid courses to not have to deal with the consequences later. (Not speaking for the community here, but just my own opinion).

That said, git-scm is not really in control by 'the git community' IMHO, but some GitHub guys. So if they will deal with the consequences later, I don't care.

@peff
Copy link
Member

peff commented Jan 25, 2017

@jnareb Thanks. I think it's still appropriate to go into the book section (though I wonder if we should pick just one of "Mastering Git" and the bundle to display, if one is a superset of the other).

@peff
Copy link
Member

peff commented Jan 25, 2017

Would it help if I create a new section for paid courses? I think it could prove to be a good resource to help many in their learning of Git.

FYI, I took this to the upstream list to get opinions (link). I'm inclined to say that yes, a new section is OK, and other people with paid tutorial content would be welcome to submit PRs to add it. If it becomes overwhelming, then we'll deal with it at that point.

@peff
Copy link
Member

peff commented Jan 25, 2017

That said, git-scm is not really in control by 'the git community' IMHO, but some GitHub guys. So if they will deal with the consequences later, I don't care.

This is not quite true anymore. The site was more or less maintainer-less as early 2016, so I've tried to pick up the slack and deal with open PRs. I have rights on the GitHub repo and the Heroku deployment. I can add people to the former if anybody wants to get involved, but I think not easily to the latter (it's bundled with and paid for as part of GitHub's Heroku deployments).

Moreover, the site has been the de-facto official Git web page for a long time, and is more officially so in the last year. As a member project of the Software Freedom Conservancy, we need it to conform to a few non-profit rules, like not promoting one commercial interest over the other (and if it fails that, we either need to fix it, or make it clear that the page is not endorsed by the project).

I plan to bring this up as a topic at the Contributor Summit next week (and on the mailing list).

@packtpartner
Copy link
Contributor Author

packtpartner commented Jan 31, 2017

I'm inclined to say that yes, a new section is OK, and other people with paid tutorial content would be welcome to submit PRs to add it.

@peff thanks for clarifying. I will make the necessary changes as suggested.

Added a new Course section to feature paid courses
@peff
Copy link
Member

peff commented Feb 17, 2017

@packtpartner Sorry, this got lost in the flurry of recent activity on the site. The courses section you added looks good, and I don't see any reason not to merge.

@peff peff merged commit 1c2f9da into git:master Feb 17, 2017
@packtpartner
Copy link
Contributor Author

Thanks a lot @peff

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