Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#921 Adds content to modules section of .app file #922

Open
wants to merge 25 commits into
base: master
Choose a base branch
from

Conversation

piosok
Copy link
Contributor

@piosok piosok commented Oct 12, 2020

No description provided.

@ignatov
Copy link
Owner

ignatov commented Nov 15, 2020

Could you please tell why this change is important?

@piosok
Copy link
Contributor Author

piosok commented Nov 23, 2020

I'm maintaining an app, which uses https://github.com/K2InformaticsGmbH/imem and https://github.com/K2InformaticsGmbH/dderl. These modules rely on .app file having modules section populated, otherwise they don't start correctly in debugger/run from intellij-erlang IDE.
As rebar3 normally automatically populates modules in .app I assumed that the right thing to do would be if compilation in intellij-erlang did the same.

@ignatov
Copy link
Owner

ignatov commented Feb 24, 2023

Hello! Can I apply the whole diff as a single commit? I'd prefer to keep the history clean.

@piosok
Copy link
Contributor Author

piosok commented Feb 26, 2023

Yes, sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants