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

Reorganize and clarify title markers section of RC #4274

Merged
merged 7 commits into from Oct 6, 2020

Conversation

hypercyte
Copy link
Contributor

@hypercyte hypercyte commented Sep 30, 2020


  • Renames title "tags" to "markers"
  • Clarifies that unofficial cuts roughly matching official cuts should use the official markers
  • Specifies how to handle multiple markers in a title

@cl8n
Copy link
Member

cl8n commented Sep 30, 2020

can you explain more what d171a4c is supposed to do, I don't see how the meaning would be any different (what other track is there?)

@hypercyte
Copy link
Contributor Author

image

was thinking of a way to word it then noffy explained it better :)

@cl8n
Copy link
Member

cl8n commented Oct 2, 2020

i think instead u should move L94 under unofficial cuts, and reword it to be clear that you don't need Cut Ver if u have an official-ish marker for that case

@cl8n cl8n changed the title Clarification on the new (Cut Ver.) rule Clarify when "Cut Ver." is required in RC Oct 2, 2020
@NoffyNoffeh
Copy link
Contributor

NoffyNoffeh commented Oct 6, 2020

@cl8n I feel like that solution would be a bit more convoluted as it then requires reading both rules to understand one, rather than being able to be read mostly on their own and understood.

@cl8n
Copy link
Member

cl8n commented Oct 6, 2020

don't really see why 94 belongs in its current place at all with this unofficial cuts rule existing, because 94 is all about unofficial cuts

in fact if you didn't read the note under 93, which doesn't seem to apply to unofficial cuts, you wouldn't know to add your own gamever/etc markers instead 🤔

ok figured this out in DM with noffy cuz my wording sucked here

Copy link
Contributor

@NoffyNoffeh NoffyNoffeh left a comment

Choose a reason for hiding this comment

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

After clarifying with clayton what he meant, here is a set of suggestions to address the issue.

wiki/Ranking_Criteria/en.md Outdated Show resolved Hide resolved
wiki/Ranking_Criteria/en.md Outdated Show resolved Hide resolved
wiki/Ranking_Criteria/en.md Outdated Show resolved Hide resolved
@hypercyte
Copy link
Contributor Author

👍👍👍👍

@pull-request-size pull-request-size bot added size/S and removed size/XS labels Oct 6, 2020
@cl8n cl8n changed the title Clarify when "Cut Ver." is required in RC Reorganize and clarify title markers section of RC Oct 6, 2020
@cl8n cl8n merged commit f4b9c31 into ppy:master Oct 6, 2020
@hypercyte hypercyte deleted the new-rule-clarification branch March 1, 2021 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants