Skip to content

Conversation

@MarkoMin
Copy link
Contributor

@MarkoMin MarkoMin commented Dec 2, 2025

Some defensiveness. Besides module duplication performs other structural checks.

Closes #10427

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

CT Test Results

    2 files     72 suites   1h 7m 10s ⏱️
1 687 tests 1 341 ✅ 344 💤 2 ❌
1 939 runs  1 535 ✅ 402 💤 2 ❌

For more details on these failures, see this check.

Results for commit a9ef3e8.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@MarkoMin
Copy link
Contributor Author

MarkoMin commented Dec 2, 2025

Hmm, looks like {transient, 1} module entry was legal before (see test application_SUITE:otp_2718/1). Should this be still treated as legal?

@IngelaAndin IngelaAndin added the team:VM Assigned to OTP team VM label Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team:VM Assigned to OTP team VM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add tests to prevent duplicate module names in app.src files

2 participants