Remove -j-std

[smondet]

Cf. issue #412

PR checklist

• New code has tests to catch future regressions
• Documentation is up-to-date
• CHANGES.md is up-to-date

[smondet]

 $ dune exec atdgen/bin/ag_main.exe -- --help | head -n 20
Generate OCaml code offering:         
  * OCaml type definitions translated from ATD file (-t)
  * serializers and deserializers for Biniou (-b)
  * serializers and deserializers for JSON (-j)
  * record-creating functions supporting default fields (-v)
  * user-specified data validators (-v)

Recommended usage: _build/default/atdgen/bin/ag_main.exe (-t|-b|-j|-v|-dep|-list|-mel) example.atd

Some options are deprecated. Use the environment variable `ATDGEN_FAIL_DEPRECATED_OPTIONS=true`
to make their use fail and hence help clean-up your build scripts.

  -type-conv 
    GEN1,GEN2,...
         Insert 'with GEN1, GEN2, ...' after OCaml type definitions for the
         type-conv preprocessor
    
  -deriving-conv 
    GEN1,GEN2,...
         Insert '[@@deriving GEN1,GEN2,...]' after OCaml type definitions for

[smondet]

Building the repo with export ATDGEN_FAIL_DEPRECATED_OPTIONS=true for now gives:

File "atdgen/test/dune", line 93, characters 0-232:
93 | (rule
94 |  (targets test_ambiguous_record_j.ml test_ambiguous_record_j.mli)
95 |  (deps    test_ambiguous_record.atd)
96 |  (action
97 |   (run %{bin:atdgen} -json -j-gen-modules -o test_ambiguous_record_j -open Test_ambiguous_record_t -ntd %{deps})))
Error: option "-json" is forbidden.
File "atdgen/test/dune", line 117, characters 0-197:
117 | (rule
118 |  (targets test_polymorphic_wrap_j.ml test_polymorphic_wrap_j.mli)
119 |  (deps    test_polymorphic_wrap.atd)
120 |  (action
121 |   (run %{bin:atdgen} -json -j-gen-modules -o test_polymorphic_wrap_j %{deps})))
Error: option "-json" is forbidden.
File "atdgen/test/dune", line 163, characters 0-139:
163 | (rule
164 |  (targets testjstd_j.ml testjstd_j.mli)
165 |  (deps    test.atd)
166 |  (action  (run %{bin:atdgen} -extend Test test.atd -json -o testjstd_j)))
Error: option "-json" is forbidden.
File "atdgen/test/dune", line 178, characters 0-128:
178 | (rule
179 |  (targets testv.ml testv.mli)
180 |  (deps    test.atd)
181 |  (action  (run %{bin:atdgen} -validate -extend Test test.atd -o testv)))
Error: option "-validate" is forbidden.
File "atdgen/test/dune", line 31, characters 0-256:
31 | (rule
32 |  (targets testj.ml testj.mli)
33 |  (deps    (:atd test.atd) test.ml test.mli)
34 |  (action
35 |   (run
36 |    %{bin:atdgen} -json -extend Test -j-custom-fields
37 |    "fun loc s -> Printf.eprintf \"Warning: skipping field %s (def: %s)\n\" s loc"
38 |    %{atd}
39 |     -o testj)))
Error: option "-json" is forbidden.

[mjambon]

Great.

[mjambon]

Looks good to me. Thank you!

[smondet]

@mjambon should we do anything special with PRs that target the "3.0" major version (ie breaking changes we may not want in a minor release)?