sync: deprecated exercises are included in output

[junedev]

I ran configlet sync in the JavaScript track and it reported

[warn] accumulate: missing 6 test cases
       - accumulate empty (64d97c14-36dd-44a8-9621-2cecebd6ed23)
       - accumulate squares (00008ed2-4651-4929-8c08-8b4dbd70872e)
       - accumulate upcases (551016da-4396-4cae-b0ec-4c3a1a264125)
       - accumulate reversed strings (cdf95597-b6ec-4eac-a838-3480d13d0d05)
       - accumulate recursively (bee8e9b6-b16f-4cd2-be3b-ccf7457e50bb)
       - accumulate recursively (0b357334-4cad-49e1-a741-425202edfc7c)

But the accumulate exercise is depreacted, both in the problem spec repo and in the JavaScript track config.json.

      {
        "slug": "accumulate",
        "name": "Accumulate",
        "uuid": "6ebe247c-3d11-48b7-8e6f-39f98359d233",
        "practices": [],
        "prerequisites": [],
        "difficulty": 5,
        "topics": ["algorithms", "callbacks", "loops", "lists"],
        "status": "deprecated"
      },

So it looked like a bug too me that it is still reported as "not in sync".

Configlet version: 4.0.0-alpha.36

[ErikSchierboom]

I've moved this to the configlet repo.

[junedev]

Oopsie, that's where I wanted to post this. Thanks @ErikSchierboom!

[ee7]

I guess configlet sync should ignore an exercise that is both out of sync, and deprecated on the track. But not ignore an exercise that is only deprecated in problem-specifications.

Agreed?

Otherwise, there isn't much point in allowing a deprecated exercise to be updated in problem-specifications (which is what happened with accumulate).

[junedev]

Yes, exactly.

[ErikSchierboom]

Agreed!