Skip to content

Patch v2.1.1#39

Merged
gregorbg merged 19 commits intothewca:latestfrom
dunkOnIT:2.0.1-latest
Apr 24, 2026
Merged

Patch v2.1.1#39
gregorbg merged 19 commits intothewca:latestfrom
dunkOnIT:2.0.1-latest

Conversation

@dunkOnIT
Copy link
Copy Markdown
Contributor

@dunkOnIT dunkOnIT commented Apr 2, 2026

  • See changelog for reason for patch and minor version increments
  • Various grammar and consistency fixes

Comment thread specification.md Outdated
Comment thread versioning-policy.md Outdated
- Adding a new field, adding values to an existing enum
- **Patch**: a change which does not meaningfully alter API output - for example, bugfixes that alter incorrect behaviour to align with documented behaviour
- **Patch**: clerical changes improving spec clarity, or a change which does not meaningfully alter API output
- Correctly an inconsistency in field naming between documentation and the example of the documentation
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The grammar in this sentence seems off

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@dunkOnIT dunkOnIT marked this pull request as ready for review April 6, 2026 12:28
@dunkOnIT
Copy link
Copy Markdown
Contributor Author

dunkOnIT commented Apr 6, 2026

Added changelog entry (no need to enumerate every change - people can check the diff if needed) and marked as Ready for Review, given that I assume you've now worked your way through the spec and its changes, so no more updates are likely?

Comment thread specification.md Outdated
| Attribute | Type | Description |
| --- | --- | --- |
| `type` | `String` | Always `percent`
| `scope` | `"single"\|"average""` | Specifies if the result should be a `single` or `average`. For ParticipationRuleset, this will always be determined by the event format, and cannot be changed via a WCIF patch.. |
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

by the round format.
Also, double full stop at the end

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Addressed

Comment thread specification.md Outdated
| --- | --- | --- |
| `type` | `String` | Always `ranking`
| `type` | `String` | Always `ranking` |
| `scope` | `"single"\|"average""` | Specifies if the result should be a `single` or `average`. For ParticipationRuleset, this will always be determined by the event format, and cannot be changed via a WCIF patch.. |
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same as my other comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Addressed

Comment thread specification.md

## Version
- Number: 2.0.0
- Number: 2.1.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hmm, will wca website ever request data for wcif 2.0.0 and 2.1.1?

Maybe we just support different versions based on what we intend to return from the wca website?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hmm, I don't quite understand this - how do you mean "Maybe we just support different versions based on what we intend to return from the wca website?"

Gregor and I chatted about this the other day - we'll basically only ever return the latest and current - so if 2.0.0 gets superceded by 2.1.1 in current, we'll cease to serve 2.0.0 - except in exceptional circumstances where we might continues to serve a deprecated version for some period of time.

@dunkOnIT dunkOnIT changed the title Patch v2.0.1 Patch v2.1.1 Apr 24, 2026
@gregorbg gregorbg merged commit ad84a77 into thewca:latest Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants