| Thread overview |
|---|
February 15, 2022 [Issue 22730] master: "dmd -i" doesn't include unit tests from imported modules | ||||
|---|---|---|---|---|
| ||||
https://issues.dlang.org/show_bug.cgi?id=22730 Dlang Bot <dlang-bot@dlang.rocks> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |pull --- Comment #1 from Dlang Bot <dlang-bot@dlang.rocks> --- @MoonlightSentinel created dlang/dmd pull request #13654 "Fix 22730 - Promote imported files (-i) to root module before parsing the declarations." fixing this issue: - Fix 22730 - Promote imported files (-i) to root module before parsing... ... the declarations. DMD PR 13224 changed the parser s.t. unittests from non-root modules are skipped, i.e. not even parsed. The new behaviour didn't work as expected when combined with `-i` because it promoted imported files to root modules *after* the parser processed the entire file. This commit moves the existing checks s.t. they are applied immediatly after the module declaration was read by the parser. https://github.com/dlang/dmd/pull/13654 -- | ||||
February 16, 2022 [Issue 22730] master: "dmd -i" doesn't include unit tests from imported modules | ||||
|---|---|---|---|---|
| ||||
https://issues.dlang.org/show_bug.cgi?id=22730 Dlang Bot <dlang-bot@dlang.rocks> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #2 from Dlang Bot <dlang-bot@dlang.rocks> --- dlang/dmd pull request #13654 "Fix 22730 - Promote imported files (-i) to root module before parsing the declarations." was merged into stable: - ea623dbb38b51ab9a0d11bc0fad79d392c66de98 by MoonlightSentinel: Fix 22730 - Promote imported files (-i) to root module before parsing... ... the declarations. DMD PR 13224 changed the parser s.t. unittests from non-root modules are skipped, i.e. not even parsed. The new behaviour didn't work as expected when combined with `-i` because it promoted imported files to root modules *after* the parser processed the entire file. This commit moves the existing checks s.t. they are applied immediatly after the module declaration was read by the parser. https://github.com/dlang/dmd/pull/13654 -- | ||||
February 18, 2022 [Issue 22730] master: "dmd -i" doesn't include unit tests from imported modules | ||||
|---|---|---|---|---|
| ||||
https://issues.dlang.org/show_bug.cgi?id=22730 --- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> --- dlang/dmd pull request #13685 "Merge stable into master" was merged into master: - 1c31d85613070cc547b166b464cff75bdf27bfa2 by MoonlightSentinel: Fix 22730 - Promote imported files (-i) to root module before parsing... ... the declarations. DMD PR 13224 changed the parser s.t. unittests from non-root modules are skipped, i.e. not even parsed. The new behaviour didn't work as expected when combined with `-i` because it promoted imported files to root modules *after* the parser processed the entire file. This commit moves the existing checks s.t. they are applied immediatly after the module declaration was read by the parser. https://github.com/dlang/dmd/pull/13685 -- | ||||
Copyright © 1999-2021 by the D Language Foundation
Permalink
Reply