Skip to content

Reword to clarify streams lacking a version number #43

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 15 additions & 7 deletions tap-version-14-specification.md
Original file line number Diff line number Diff line change
Expand Up @@ -156,23 +156,31 @@ backwards compatibility.

### Version

To indicate that this is TAP14 the first line _must_ be
The TAP version of a TAP stream is indicated on the first line.

```tap
TAP version 14
```

Harnesses _may_ interpret ostensibly
[TAP13](https://testanything.org/tap-version-13-specification.html) streams
as TAP14, as this specification is compatible with observed behavior of
existing TAP13 consumers and producers. That is, they _may_ treat this as
a valid Version line while parsing TAP14:
If there is no version line on the first line, the TAP version shall be 12.

#### Backwards compatibility of TAP versions

This specification is compatible with observed behavior of existing
[TAP12](https://testanything.org/tap-specification.html) and
[TAP13](https://testanything.org/tap-version-13-specification.html) consumers
and producers.

Harnesses _should_ interpret ostensibly older TAP streams as TAP14. That is, they
_should_ treat this as a valid and supported Version line while parsing TAP14:

```tap
TAP version 13
```

Harnesses _may_ treat any TAP stream lacking a version as a failed test.
and likewise treat a missing Version line as implicitly TAP14.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought missing version line would be TAP12?


Harnesses _may_ instead treat any ostensibly older TAP stream as a failed test.

### Plan

Expand Down