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

sage-coverage expects doctests for closures #4323

Closed
robertwb opened this issue Oct 19, 2008 · 4 comments
Closed

sage-coverage expects doctests for closures #4323

robertwb opened this issue Oct 19, 2008 · 4 comments

Comments

@robertwb
Copy link
Contributor

Unfortunately, there is often no way to doctest such functions directly, and for some re-writing them as lambda functions is impossible or reduces readability.

CC: @zimmermann6

Component: doctest coverage

Issue created by migration from https://trac.sagemath.org/ticket/4323

@jhpalmieri
Copy link
Member

comment:4

Is this the same as #877?

John (who's not quite sure what "closures" are)

P.S. By the way, I think it takes a certain amount of hubris to write a program like 'sage-coverage' which itself has no doctests or documentation :)

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Feb 9, 2009

comment:5

Replying to @jhpalmieri:

Is this the same as #877?

John (who's not quite sure what "closures" are)

Not sure.

P.S. By the way, I think it takes a certain amount of hubris to write a program like 'sage-coverage' which itself has no doctests or documentation :)

Yes, this certainly registered rather strongly on my irony meter, but right now there is no coverage for the scripts in local/bin, even though most people agreed that it would be a pretty good idea. This will hopefully happen sooner or later.

Cheers,

Michael

@zimmermann6
Copy link

comment:6

Is this the same as #877?

yes.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Feb 10, 2009

comment:7

Since this is a dupe of #877 I am closing this ticket as a dupe.

If someone disagrees please reopen.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin removed this from the sage-3.4.1 milestone Feb 10, 2009
@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Feb 10, 2009
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants