forked from irungentoo/toxcore
-
Notifications
You must be signed in to change notification settings - Fork 302
Review all public API for 0.3.0 #655
Copy link
Copy link
Open
Labels
P3Low priorityLow prioritycleanupInternal code cleanup, possibly affecting semantics, e.g. deleting a deprecated feature.Internal code cleanup, possibly affecting semantics, e.g. deleting a deprecated feature.help wantedExtra attention is neededExtra attention is neededtestAdding missing tests, refactoring tests; no production code changeAdding missing tests, refactoring tests; no production code change
Milestone
Description
Metadata
Metadata
Assignees
Labels
P3Low priorityLow prioritycleanupInternal code cleanup, possibly affecting semantics, e.g. deleting a deprecated feature.Internal code cleanup, possibly affecting semantics, e.g. deleting a deprecated feature.help wantedExtra attention is neededExtra attention is neededtestAdding missing tests, refactoring tests; no production code changeAdding missing tests, refactoring tests; no production code change
I found in IRC logs that porting TES API to use APIDSL was an API breaking change at some point in late 2016, so I guess we settled on #310 which didn't break anything (or did it? I didn't check), leaving the breaking changes for 0.2.0.
In any case, it makes sense to review all public API before 0.2.0 just in case we want to change something.