Ticket #8108 (closed enhancement: fixed)

Opened 6 weeks ago

Last modified 5 weeks ago

Expand the Sage Developer Guide for newcomers

Reported by: rbeezer Owned by: mvngu
Priority: major Milestone: sage-4.3.2
Component: documentation Keywords:
Cc: mvngu, leif Author(s): Rob Beezer, Minh Van Nguyen
Report Upstream: N/A Reviewer(s): Ross Kyprianou, John Palmieri
Merged in: sage-4.3.2.rc0 Work issues:

Description

Newcomers to Sage development could use more comprehensive guidance on getting started.

A rough draft of possible new content is attached to this ticket.

Attachments

how-to-develop.txt Download (10.8 KB) - added by rbeezer 6 weeks ago.
trac_8108-beginner-guide.patch Download (18.8 KB) - added by mvngu 6 weeks ago.
based on Sage 4.3.2.alpha0
trac_8108-typos.patch Download (1.3 KB) - added by jhpalmieri 5 weeks ago.
apply on top of other patch

Change History

Changed 6 weeks ago by rbeezer

  Changed 6 weeks ago by rbeezer

  • cc mvngu added

Changed 6 weeks ago by mvngu

based on Sage 4.3.2.alpha0

  Changed 6 weeks ago by mvngu

  • status changed from new to needs_review
  • milestone changed from sage-wishlist to sage-4.3.2
  • author set to Rob Beezer, Minh Van Nguyen

I have attached a patch that includes information from  how-to-develop.txt. I also added more stuff, which means I can no longer review Rob's work. What's missing is a section on patch management using Mercurial queue. That could be for a separate ticket if necessary.

  Changed 5 weeks ago by leif

  • cc leif added

Thanks (also for disclosing Gauss' e-mail address).

follow-up: ↓ 7   Changed 5 weeks ago by rossk

  • status changed from needs_review to positive_review

This will save newbie developers (and reviewers) hours of work (includes me! :-) For what its worth, Ive read it and it has answered a dozen questions. Thanks. We should remember that any other info can be included over time. Given this stuff is so valuable IMHO and Ive already tried much of what Ive read (but now it makes sense ;-) Im going to give this a positive review, if I may, even if you guys want to finesse it before including it. AGAIN THANKS!

  Changed 5 weeks ago by jhpalmieri

Here's a little patch to fix a few typos.

Changed 5 weeks ago by jhpalmieri

apply on top of other patch

  Changed 5 weeks ago by rbeezer

John - thanks for the fixes (and the once-over).

John's fixes look good to me. Minh handled the formatting (and lots more), so hopefully this will pass for a review of John's changes. ;-)

Rob

in reply to: ↑ 4 ; follow-up: ↓ 9   Changed 5 weeks ago by rbeezer

Ross,

Thanks for the enthusiastic review! I've included you as a cc on #8147 that will expand the material on Mercurial queues at the end.

Rob

Replying to rossk:

This will save newbie developers (and reviewers) hours of work (includes me! :-) For what its worth, Ive read it and it has answered a dozen questions. Thanks. We should remember that any other info can be included over time. Given this stuff is so valuable IMHO and Ive already tried much of what Ive read (but now it makes sense ;-) Im going to give this a positive review, if I may, even if you guys want to finesse it before including it. AGAIN THANKS!

  Changed 5 weeks ago by mvngu

  • status changed from positive_review to closed
  • reviewer set to Ross Kyprianou, John Palmieri
  • resolution set to fixed
  • merged set to sage-4.3.2.rc0

Merged in this order:

  1.  trac_8108-beginner-guide.patch
  2.  trac_8108-typos.patch

in reply to: ↑ 7   Changed 5 weeks ago by rossk

Replying to rbeezer:

Ross, Thanks for the enthusiastic review! I've included you as a cc on #8147 that will expand the material on Mercurial queues at the end. Rob

No probs Rob :-)

Note: See TracTickets for help on using tickets.