Skip to content

WIP draft ranges concept #803

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

Merged
merged 17 commits into from
Mar 4, 2025
Merged

WIP draft ranges concept #803

merged 17 commits into from
Mar 4, 2025

Conversation

colinleach
Copy link
Contributor

@colinleach colinleach commented Oct 2, 2024

I split this out of the earlier Arrays draft.

The best exercise I can find for it is elixir/chessboard. We might want to expand that a bit, but it seems like a useful starting point.

Edit: Because go/chessboard is quite unlike the Elixir version, we could maybe take the most suitable features of each.

@colinleach colinleach marked this pull request as ready for review March 2, 2025 23:06
@colinleach colinleach requested a review from depial March 2, 2025 23:06
@colinleach
Copy link
Contributor Author

I think this + the Chessboard exercise are now good to review. My local Git installation got corrupted somehow, so I hope I caught all the issues.

@colinleach colinleach removed the request for review from depial March 2, 2025 23:50
@colinleach colinleach marked this pull request as draft March 2, 2025 23:50
@colinleach
Copy link
Contributor Author

colinleach commented Mar 2, 2025

The issues with config.json are getting silly. I've moved this back to draft. Once some of the earlier concepts are merged, I'll rebase onto main and have another attempt. At the moment, there are a lot of concept PRs open, and some of them are reusing old UUIDs - all very confusing.

I really wish we could have deleted all the old learning track stuff and started again from scratch, but Jeremy freaked when I suggested it. The link between GH and the Exercism website is complex and sometimes brittle, plus I used up all my credibility back in 2023 when I broke the R track with some over-enthusiastic changes (very embarrassing, and I'm keen to avoid a repeat).

@colinleach colinleach marked this pull request as ready for review March 4, 2025 00:22
@colinleach
Copy link
Contributor Author

With so many earlier concepts now merged, adding this one to config.json is now easier. Not that that stopped me messing it up a couple of time!

Copy link
Contributor

@depial depial left a comment

Choose a reason for hiding this comment

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

Only minor nits, which need not be addressed (however "an vector" should probably at least be made "a vector"

depial added 2 commits March 4, 2025 11:22
Sync with introduction.md
Sync with about.md
@depial depial merged commit 6fef8a4 into exercism:main Mar 4, 2025
1 check passed
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