-
Notifications
You must be signed in to change notification settings - Fork 74
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
Fixes #19658: Update elm to 0.19.1 #3767
Fixes #19658: Update elm to 0.19.1 #3767
Conversation
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
for PROJECT in ${PROJECTS[*]}; do | ||
cd ${ELM_DIR}/${PROJECT} | ||
elm make --optimize sources/rudder-${PROJECT}.elm --output=generated/rudder-${PROJECT}.js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know how it could work ad rudder-notifications.elm was not in a sources directory so maybe I'm misunderstanding something
PR updated with a new commit |
PR updated with a new commit |
OK, squash merging this PR |
7e7a645
to
4722c64
Compare
https://issues.rudder.io/issues/19658