Navigation Menu

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

Jump to bottom

API design and documentation review (whole public API) #37

Closed
GoogleCodeExporter opened this issue Mar 15, 2015 · 9 comments
Closed

API design and documentation review (whole public API) #37

GoogleCodeExporter opened this issue Mar 15, 2015 · 9 comments
Labels
documentation release blocking
Milestone
1.0.0

Comments

@GoogleCodeExporter
Copy link 

Before we hit v1, we should go over the whole documented API, member by member, 
and make sure it makes sense, has appropriate names, doesn't expose anything we 
don't want etc.

Original issue reported on code.google.com by jonathan.skeet on 7 Feb 2012 at 8:09
@GoogleCodeExporter
Copy link
Author 

I've completed the initial pass through the main NodaTime namespace, and filed 
a ton of issues.  The other namespaces are still outstanding (and it'd probably 
benefit from a final scan, too).

Original comment by malcolm.rowe on 27 Jul 2012 at 5:29

  • Changed state: Started

@GoogleCodeExporter
Copy link
Author

Original comment by malcolm.rowe on 30 Jul 2012 at 7:26

  • Added labels: Milestone-1.0
  • Removed labels: V1-Blocker

@GoogleCodeExporter
Copy link
Author

Original comment by malcolm.rowe on 30 Jul 2012 at 7:49

  • Added labels: Type-Docs

@GoogleCodeExporter
Copy link
Author

Original comment by malcolm.rowe on 23 Oct 2012 at 12:48

  • Added labels: Priority-Critical

@GoogleCodeExporter
Copy link
Author 

NodaTime.Calendars is done.

Original comment by malcolm.rowe on 24 Oct 2012 at 6:00

@GoogleCodeExporter
Copy link
Author 

For the sake of tracking:

- NodaTime.Globalization is done (by virtue of having been removed)
- NodaTime.Serialization.JsonNet is being ignored.
- NodaTime.Testing is done.
- NodaTime.Testing.TimeZones is done.
- NodaTime.Text is done.

I have a miscellany of clarifications and touchups from the above (and a few 
for NodaTime and NodaTime.Calendars), but nothing that affects the API so far 
other than issue 137 and issue 138.

Original comment by malcolm.rowe on 2 Nov 2012 at 9:48

@GoogleCodeExporter
Copy link
Author

Original comment by malcolm.rowe on 10 Nov 2012 at 10:20

  • Added labels: Milestone-1.0.0
  • Removed labels: Milestone-1.0

@GoogleCodeExporter
Copy link
Author 

Probably okay to close this one now, Malcolm?

Original comment by jonathan.skeet on 25 Dec 2012 at 12:15

@GoogleCodeExporter
Copy link
Author 

I'm actually still not completely done from the initial pass.  We can close if 
you like, but I'm still doing this in my copious free time ;-)

Original comment by malcolm.rowe on 25 Dec 2012 at 8:28

@malcolmr malcolmr modified the milestone: 1.0.0 Mar 15, 2015
@jskeet jskeet closed this as completed Mar 15, 2017
malcolmr pushed a commit that referenced this issue Apr 26, 2020
@jskeet
Check member documentation, but only for unstable releases
78d7163 
Fixes #37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation release blocking
Projects
None yet
Milestone
1.0.0
Development

No branches or pull requests
3 participants
@jskeet @malcolmr @GoogleCodeExporter