fixed tree-shaking language exporter may list entities twice

[enikao]

No description provided.

[github-actions]

Test Results

695 tests  +1   695 ✅ +1   11s ⏱️ -1s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 815ee0c. ± Comparison against base commit de6b089.

♻️ This comment has been updated with latest results.

[dslmeinte]

Looks good. The true eating of the pudding happens after migrating to 2021.3, of course.

One thing: the TestDeps-*.json files seem to be missing LionCore-builtins in their top-level languages KV-pair. (Not sure how many, but at least the new one.) Not a problem but also not according to spec. (Could fix that later in a separate PR.)

[enikao]

One thing: the TestDeps-*.json files seem to be missing LionCore-builtins in their top-level languages KV-pair. (Not sure how many, but at least the new one.) Not a problem but also not according to spec. (Could fix that later in a separate PR.)

Really?

For me, the first lines of solutions/io.lionweb.mps.json.test/resources/TestDeps-AvoidMultipleConcepts.json are:

{
  "serializationFormatVersion": "2023.1",
  "languages": [
    {
      "key": "LionCore-M3",
      "version": "2023.1"
    },
    {
      "key": "LionCore-builtins",
      "version": "2023.1"
    }
  ],
  "nodes": [

What's the issue there?

[dslmeinte]

What's the issue there?

The issue I was looking at only commit 367c6524 at this point, and didn't see that commit 9112a87c fixed it. Sorry about that!