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

update Developers' Guide to reflect new process for working on tickets #7944

Closed
sagetrac-mvngu mannequin opened this issue Jan 16, 2010 · 6 comments
Closed

update Developers' Guide to reflect new process for working on tickets #7944

sagetrac-mvngu mannequin opened this issue Jan 16, 2010 · 6 comments
Assignees
@sagetrac-mvngu
Labels
c: documentation p: minor / 4 t: bug
Milestone
sage-4.3.3

Comments

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jan 16, 2010

From sage-devel:

It seems to me that developers guide is a bit out of date (e.g. it 
has guidelines about adding [with patch, needs review] to tickets 
on trac)

This ticket is a prerequisite for #8199.

Component: documentation

Author: Minh Van Nguyen

Reviewer: Andrey Novoseltsev

Merged: sage-4.3.3.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/7944
@sagetrac-mvngu sagetrac-mvngu mannequin added this to the sage-4.3.3 milestone Jan 16, 2010
@sagetrac-mvngu sagetrac-mvngu mannequin self-assigned this Jan 16, 2010
@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented Feb 5, 2010

Attachment: trac_7944-dev-guide.patch.gz

based on Sage 4.3.2.rc0

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented Feb 5, 2010

Author: Minh Van Nguyen

@sagetrac-mvngu sagetrac-mvngu mannequin added the s: needs review label Feb 5, 2010
@sagetrac-mvngu

This comment has been minimized.

Sign in to view
@novoselt
Copy link
Member

novoselt commented Feb 6, 2010

comment:3

Looks good to me and documentation builds. Thanks for all the work you have done on documenting the development process!!!

@novoselt
Copy link
Member

novoselt commented Feb 6, 2010

Reviewer: Andrey Novoseltsev

@qed777
Copy link
Mannequin

qed777 mannequin commented Feb 11, 2010

Merged: sage-4.3.3.alpha0

@qed777 qed777 mannequin removed the s: positive review label Feb 11, 2010
@qed777 qed777 mannequin closed this as completed Feb 11, 2010
@sagetrac-mvngu sagetrac-mvngu mannequin mentioned this issue Feb 6, 2010
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-mvngu sagetrac-mvngu

Labels
c: documentation p: minor / 4 t: bug
Projects
None yet
Milestone
sage-4.3.3
Development

No branches or pull requests
1 participant
@novoselt