Skip to content

Commit bdd34a3

Browse files
jimschubertwing328
authored andcommitted
[ci] Removes elm from ensure-up-to-date (#4471)
bin/elm-petstore-all.sh invokes elm-petstore.sh and elm-0.18-petstore.sh. Both of these define `ELM_POST_PROCESS_FILE` for post-processing the generated files. If a user doesn't have elm-format installed, they may not realize that ensure-up-to-date has failed which causes CI to fail due to differences in the ELM generated outputs. This confusion can lead to a lot of downtime for contributors. For example, I encountered this while adding feature set information to all generators. I thought I had introduced the error and spent too long looking through my changeset and re-running `ensure-up-to-date` in the background before noticing the failed output. I was able to generate proper output by installing elm-format. With 80+ languages/frameworks and a rule for contributors to unblock CI by re-generating any failed samples, it's not feasible (in some cases, not possible) to ask contributors to install tooling specific post-processors. We'll have to rely on elm contributors to run the script manually. Ideally, elm generator templates should be updated to have properly formatted outputs as a default. We may want to consider documenting standards of what we put in the scripts under bin/*sh and bin/utils/ensure-up-to-date, one of those standards being that we omit toolchain specific post-processors.
1 parent fa1e003 commit bdd34a3

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

bin/utils/ensure-up-to-date

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ declare -a scripts=(
6868
"./bin/dart-petstore.sh"
6969
"./bin/dart2-petstore.sh"
7070
"./bin/java-play-framework-petstore-server-all.sh"
71-
"./bin/elm-petstore-all.sh"
71+
#"./bin/elm-petstore-all.sh"
7272
"./bin/meta-codegen.sh"
7373
# OTHERS
7474
"./bin/utils/export_docs_generators.sh"

0 commit comments

Comments
 (0)