Updated darts test.toml to v3 format

Programming Kotlin
1

PR: https://github.com/exercism/kotlin/pull/730

Initial forum post about updating to v3 format

Updates tests.toml for darts to the v3 format so configlet doesn’t report missing tests

[no important files changed]

As I understand it, it’s preferred to make a forum post about PRs to a track, and I’ve been advised to make a PR for each exercise sync.

Should I consolidate those into one Forum post? I feel like it would clog the forum a bit after doing a few of these.

2

Hmm, if all you’re doing is bin/configure sync --tests then I’d just make one PR to sync them all.

3

BTW, the “no important files changed” line goes into the merge message, not the PR description.

4

attn @SleeplessByte

1 Like
5

Thanks @glennj

6

To be honest, if there is more work to be done, I’d rather have a separate PR per exercise in case of discussion holding back a merge.

7

I regenerated 🚨 configlet sync --test found unsynced tests · Issue #731 · exercism/kotlin · GitHub.

Can you comment on it so I can assign you to that? I would like the following:

  • No need to open a new Forum post for each PR, you now have approval to PR anything in that list.
  • Collate all exercises that ONLY need a sync (so no code change)
  • For each exercise that needs a code change, open a separate PR
  • Link each PR in that issue (so I can easily open them when I have time)

You will gain some rep for doing this work.

Do not worry about the auto-closer, and feel free to profusely ping me here or on GitHub. I get a lot of notifications so double tagging works best (like Glenn did!)

image

3 Likes
8

This makes sense.

I’ve commented on the issue in GitHub, and I’ll ping if I have any questions.

1 Like
9

Noted. Thanks for pointing that out

1 Like