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

Add link about migration to Cubing Contests & fix two inconsistencies in the results #15

Merged
merged 2 commits into from
Nov 16, 2023

Conversation

dmint789
Copy link
Contributor

  1. Replace the link to submit more results with a link to Cubing Contests with a message mentioning that the project has migrated.
  2. Fix round format for Mirror Blocks at Munich Open 2010 to match the order of the results.
  3. Reverse the rounds for Mirror Blocks at Kanazawa Open 2008. It is assumed that they were in the original order by mistake, since round 1 had half the number of people in finals, and it was Ao5, while finals was Bo2, which makes little sense.

@@ -1,7 +1,7 @@
{
"formatter": {
"indentStyle": "space",
"indentSize": 2,
"indentWidth": 2,
Copy link
Member

Choose a reason for hiding this comment

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

Hmm, this shouldn't be causing a linter issue, but worth doing anyhow. I'll do that in a separate commit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It threw an error before, so I thought I'd try to fix it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should I undo this?

Copy link
Member

Choose a reason for hiding this comment

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

Actually, it looks like we can't switch to this yet, Biome is still in the process of switching.

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, okay, I've made indentWidth work now.

@lgarron lgarron merged commit 46c19b2 into cubing:main Nov 16, 2023
1 check failed
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.

2 participants