docs: scrub stale v0/v1 markers from shipping comments#123
Open
UnsignedChad wants to merge 1 commit into
Open
Conversation
UnsignedChad
force-pushed
the
chore/v0-comment-scrub
branch
3 times, most recently
from
87ba0bf to
600d410
Compare
dozens of comments labelled current behaviour as v0 or v1 with no roadmap signposting what v2 would change. dropped the label or substituted with default / current / baseline. Touchstone keeps v1/v2 (real spec versions), mpkit DESIGN keeps roadmap labels, fft_test keeps the v0/v1 variable names.
UnsignedChad
force-pushed
the
chore/v0-comment-scrub
branch
from
600d410 to
2753f43
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Audit fix 7 of 8. ~30 comments through sikit + emikit labelled current behaviour as v0 or v1 with no roadmap behind the label. Either dropped the marker or substituted with default / current / baseline. Kept:
Also fixed one user-visible AmiLoadError message (was not implemented in v0, now not implemented) and a handful of en-dashes the user dislikes that snuck in via copy-paste from old docs.
717 tests still pass.