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.

Sign up for GitHub

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

Jump to bottom

morphir-elm should fail if an exposed module does not exist #1172

Open
duane opened this issue Aug 8, 2024 · 1 comment
Open

morphir-elm should fail if an exposed module does not exist #1172

duane opened this issue Aug 8, 2024 · 1 comment

Comments

@duane
Copy link

duane commented Aug 8, 2024

Is your feature request related to a problem? Please describe.

I spent a good deal of time trying to debug why my type wasn't showing up in a generated IR file only to find I had exposed the wrong path to the module.

Describe the solution you'd like

Ideally, morphir-elm would verify that all exposed modules actually exist, and fail if they do not.

Describe alternatives you've considered

A warning would be fine, too.

🥂
@duane duane changed the title morphir-elm should fail if an exposed path does not exist morphir-elm should fail if an exposed module does not exist Aug 8, 2024
AttilaMihaly added a commit to AttilaMihaly/morphir-elm that referenced this issue Nov 13, 2024
@AttilaMihaly
Report invalid exposed modules. finos#1172
55054c9
@AttilaMihaly AttilaMihaly mentioned this issue Nov 13, 2024
Merged
AttilaMihaly added a commit to AttilaMihaly/morphir-elm that referenced this issue Nov 13, 2024
@AttilaMihaly
Fixed codec. finos#1172
96e5f38
AttilaMihaly added a commit to AttilaMihaly/morphir-elm that referenced this issue Nov 13, 2024
@AttilaMihaly
Add accidentally removed entry. finos#1172
3ceb97d
AttilaMihaly added a commit to AttilaMihaly/morphir-elm that referenced this issue Nov 26, 2024
@AttilaMihaly
Fix the tests. finos#1172
590689d
AttilaMihaly added a commit to AttilaMihaly/morphir-elm that referenced this issue Dec 12, 2024
@AttilaMihaly
Fix security issues. finos#1172
7e914e5
@AttilaMihaly
Copy link
Member

Hi @duane, thanks for reporting the issue! I just merged #1205 which fixes the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@duane @AttilaMihaly