Skip to content

Commit a0a637d

Browse files
committed
refactor a bit and add transform for dropping unit args in apply
1 parent 64aaa9f commit a0a637d

File tree

4 files changed

+191
-147
lines changed

4 files changed

+191
-147
lines changed

packages/@rescript/runtime/Js_date.res

Lines changed: 52 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,8 @@ let november1 = Js.Date.makeWithYM(~year=2020.0, ~month=10.0, ())
127127
*/
128128
@deprecated({
129129
reason: "Use `Date.makeWithYM` instead.",
130-
migrate: Date.makeWithYM(
130+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
131+
Date.makeWithYM(
131132
~year=Float.toInt(%insert.labelledArgument("year")),
132133
~month=Float.toInt(%insert.labelledArgument("month")),
133134
),
@@ -144,7 +145,8 @@ on MDN.
144145
*/
145146
@deprecated({
146147
reason: "Use `Date.makeWithYMD` instead.",
147-
migrate: Date.makeWithYMD(
148+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
149+
Date.makeWithYMD(
148150
~year=Float.toInt(%insert.labelledArgument("year")),
149151
~month=Float.toInt(%insert.labelledArgument("month")),
150152
~day=Float.toInt(%insert.labelledArgument("date")),
@@ -162,7 +164,8 @@ on MDN. Fractional parts of the arguments are ignored.
162164
*/
163165
@deprecated({
164166
reason: "Use `Date.makeWithYMDH` instead.",
165-
migrate: Date.makeWithYMDH(
167+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
168+
Date.makeWithYMDH(
166169
~year=Float.toInt(%insert.labelledArgument("year")),
167170
~month=Float.toInt(%insert.labelledArgument("month")),
168171
~day=Float.toInt(%insert.labelledArgument("date")),
@@ -182,7 +185,8 @@ on MDN.
182185
*/
183186
@deprecated({
184187
reason: "Use `Date.makeWithYMDHM` instead.",
185-
migrate: Date.makeWithYMDHM(
188+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
189+
Date.makeWithYMDHM(
186190
~year=Float.toInt(%insert.labelledArgument("year")),
187191
~month=Float.toInt(%insert.labelledArgument("month")),
188192
~day=Float.toInt(%insert.labelledArgument("date")),
@@ -223,7 +227,8 @@ Js.Date.makeWithYMDHMS(
223227
*/
224228
@deprecated({
225229
reason: "Use `Date.makeWithYMDHMS` instead.",
226-
migrate: Date.makeWithYMDHMS(
230+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
231+
Date.makeWithYMDHMS(
227232
~year=Float.toInt(%insert.labelledArgument("year")),
228233
~month=Float.toInt(%insert.labelledArgument("month")),
229234
~day=Float.toInt(%insert.labelledArgument("date")),
@@ -258,7 +263,8 @@ let november1 = Js.Date.utcWithYM(~year=2020.0, ~month=10.0, ())
258263
*/
259264
@deprecated({
260265
reason: "Use `Date.UTC.makeWithYM` instead.",
261-
migrate: Date.UTC.makeWithYM(
266+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
267+
Date.UTC.makeWithYM(
262268
~year=Float.toInt(%insert.labelledArgument("year")),
263269
~month=Float.toInt(%insert.labelledArgument("month")),
264270
),
@@ -275,7 +281,8 @@ on MDN.
275281
*/
276282
@deprecated({
277283
reason: "Use `Date.UTC.makeWithYMD` instead.",
278-
migrate: Date.UTC.makeWithYMD(
284+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
285+
Date.UTC.makeWithYMD(
279286
~year=Float.toInt(%insert.labelledArgument("year")),
280287
~month=Float.toInt(%insert.labelledArgument("month")),
281288
~day=Float.toInt(%insert.labelledArgument("date")),
@@ -294,7 +301,8 @@ on MDN.
294301
*/
295302
@deprecated({
296303
reason: "Use `Date.UTC.makeWithYMDH` instead.",
297-
migrate: Date.UTC.makeWithYMDH(
304+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
305+
Date.UTC.makeWithYMDH(
298306
~year=Float.toInt(%insert.labelledArgument("year")),
299307
~month=Float.toInt(%insert.labelledArgument("month")),
300308
~day=Float.toInt(%insert.labelledArgument("date")),
@@ -314,7 +322,8 @@ on MDN.
314322
*/
315323
@deprecated({
316324
reason: "Use `Date.UTC.makeWithYMDHM` instead.",
317-
migrate: Date.UTC.makeWithYMDHM(
325+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
326+
Date.UTC.makeWithYMDHM(
318327
~year=Float.toInt(%insert.labelledArgument("year")),
319328
~month=Float.toInt(%insert.labelledArgument("month")),
320329
~day=Float.toInt(%insert.labelledArgument("date")),
@@ -343,7 +352,8 @@ on MDN.
343352
*/
344353
@deprecated({
345354
reason: "Use `Date.UTC.makeWithYMDHMS` instead.",
346-
migrate: Date.UTC.makeWithYMDHMS(
355+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
356+
Date.UTC.makeWithYMDHMS(
347357
~year=Float.toInt(%insert.labelledArgument("year")),
348358
~month=Float.toInt(%insert.labelledArgument("month")),
349359
~day=Float.toInt(%insert.labelledArgument("date")),
@@ -797,7 +807,8 @@ future == Js.Date.getTime(date1)
797807
*/
798808
@deprecated({
799809
reason: "Use `Date.setFullYearM` instead.",
800-
migrate: Date.setFullYearM(
810+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
811+
Date.setFullYearM(
801812
~year=Float.toInt(%insert.labelledArgument("year")),
802813
~month=Float.toInt(%insert.labelledArgument("month")),
803814
),
@@ -825,7 +836,8 @@ future == Js.Date.getTime(date1)
825836
@send
826837
@deprecated({
827838
reason: "Use `Date.setFullYearMD` instead.",
828-
migrate: Date.setFullYearMD(
839+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
840+
Date.setFullYearMD(
829841
~year=Float.toInt(%insert.labelledArgument("year")),
830842
~month=Float.toInt(%insert.labelledArgument("month")),
831843
~day=Float.toInt(%insert.labelledArgument("date")),
@@ -876,7 +888,8 @@ futureTime == Js.Date.getTime(date1)
876888
*/
877889
@deprecated({
878890
reason: "Use `Date.setHoursM` instead.",
879-
migrate: Date.setHoursM(
891+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
892+
Date.setHoursM(
880893
~hours=Float.toInt(%insert.labelledArgument("hours")),
881894
~minutes=Float.toInt(%insert.labelledArgument("minutes")),
882895
),
@@ -904,7 +917,8 @@ futureTime == Js.Date.getTime(date1)
904917
@send
905918
@deprecated({
906919
reason: "Use `Date.setHoursMS` instead.",
907-
migrate: Date.setHoursMS(
920+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
921+
Date.setHoursMS(
908922
~hours=Float.toInt(%insert.labelledArgument("hours")),
909923
~minutes=Float.toInt(%insert.labelledArgument("minutes")),
910924
~seconds=Float.toInt(%insert.labelledArgument("seconds")),
@@ -940,7 +954,8 @@ futureTime == Js.Date.getTime(date1)
940954
@send
941955
@deprecated({
942956
reason: "Use `Date.setHoursMSMs` instead.",
943-
migrate: Date.setHoursMSMs(
957+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
958+
Date.setHoursMSMs(
944959
~hours=Float.toInt(%insert.labelledArgument("hours")),
945960
~minutes=Float.toInt(%insert.labelledArgument("minutes")),
946961
~seconds=Float.toInt(%insert.labelledArgument("seconds")),
@@ -1023,7 +1038,8 @@ futureTime == Js.Date.getTime(date1)
10231038
@send
10241039
@deprecated({
10251040
reason: "Use `Date.setMinutesS` instead.",
1026-
migrate: Date.setMinutesS(
1041+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1042+
Date.setMinutesS(
10271043
~minutes=Float.toInt(%insert.labelledArgument("minutes")),
10281044
~seconds=Float.toInt(%insert.labelledArgument("seconds")),
10291045
),
@@ -1050,7 +1066,8 @@ futureTime == Js.Date.getTime(date1)
10501066
@send
10511067
@deprecated({
10521068
reason: "Use `Date.setMinutesSMs` instead.",
1053-
migrate: Date.setMinutesSMs(
1069+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1070+
Date.setMinutesSMs(
10541071
~minutes=Float.toInt(%insert.labelledArgument("minutes")),
10551072
~seconds=Float.toInt(%insert.labelledArgument("seconds")),
10561073
~milliseconds=Float.toInt(%insert.labelledArgument("milliseconds")),
@@ -1145,7 +1162,8 @@ futureTime == Js.Date.getTime(date1)
11451162
@send
11461163
@deprecated({
11471164
reason: "Use `Date.setSecondsMs` instead.",
1148-
migrate: Date.setSecondsMs(
1165+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1166+
Date.setSecondsMs(
11491167
~seconds=Float.toInt(%insert.labelledArgument("seconds")),
11501168
~milliseconds=Float.toInt(%insert.labelledArgument("milliseconds")),
11511169
),
@@ -1237,7 +1255,8 @@ future == Js.Date.getTime(date1)
12371255
@send
12381256
@deprecated({
12391257
reason: "Use `Date.setUTCFullYearM` instead.",
1240-
migrate: Date.setUTCFullYearM(
1258+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1259+
Date.setUTCFullYearM(
12411260
~year=Float.toInt(%insert.labelledArgument("year")),
12421261
~month=Float.toInt(%insert.labelledArgument("month")),
12431262
),
@@ -1264,7 +1283,8 @@ future == Js.Date.getTime(date1)
12641283
@send
12651284
@deprecated({
12661285
reason: "Use `Date.setUTCFullYearMD` instead.",
1267-
migrate: Date.setUTCFullYearMD(
1286+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1287+
Date.setUTCFullYearMD(
12681288
~year=Float.toInt(%insert.labelledArgument("year")),
12691289
~month=Float.toInt(%insert.labelledArgument("month")),
12701290
~day=Float.toInt(%insert.labelledArgument("date")),
@@ -1315,7 +1335,8 @@ futureTime == Js.Date.getTime(date1)
13151335
@send
13161336
@deprecated({
13171337
reason: "Use `Date.setUTCHoursM` instead.",
1318-
migrate: Date.setUTCHoursM(
1338+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1339+
Date.setUTCHoursM(
13191340
~hours=Float.toInt(%insert.labelledArgument("hours")),
13201341
~minutes=Float.toInt(%insert.labelledArgument("minutes")),
13211342
),
@@ -1343,7 +1364,8 @@ futureTime == Js.Date.getTime(date1)
13431364
@send
13441365
@deprecated({
13451366
reason: "Use `Date.setUTCHoursMS` instead.",
1346-
migrate: Date.setUTCHoursMS(
1367+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1368+
Date.setUTCHoursMS(
13471369
~hours=Float.toInt(%insert.labelledArgument("hours")),
13481370
~minutes=Float.toInt(%insert.labelledArgument("minutes")),
13491371
~seconds=Float.toInt(%insert.labelledArgument("seconds")),
@@ -1379,7 +1401,8 @@ futureTime == Js.Date.getTime(date1)
13791401
@send
13801402
@deprecated({
13811403
reason: "Use `Date.setUTCHoursMSMs` instead.",
1382-
migrate: Date.setUTCHoursMSMs(
1404+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1405+
Date.setUTCHoursMSMs(
13831406
~hours=Float.toInt(%insert.labelledArgument("hours")),
13841407
~minutes=Float.toInt(%insert.labelledArgument("minutes")),
13851408
~seconds=Float.toInt(%insert.labelledArgument("seconds")),
@@ -1460,7 +1483,8 @@ futureTime == Js.Date.getTime(date1)
14601483
@send
14611484
@deprecated({
14621485
reason: "Use `Date.setUTCMinutesS` instead.",
1463-
migrate: Date.setUTCMinutesS(
1486+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1487+
Date.setUTCMinutesS(
14641488
~minutes=Float.toInt(%insert.labelledArgument("minutes")),
14651489
~seconds=Float.toInt(%insert.labelledArgument("seconds")),
14661490
),
@@ -1493,7 +1517,8 @@ futureTime == Js.Date.getTime(date1)
14931517
@send
14941518
@deprecated({
14951519
reason: "Use `Date.setUTCMinutesSMs` instead.",
1496-
migrate: Date.setUTCMinutesSMs(
1520+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1521+
Date.setUTCMinutesSMs(
14971522
~minutes=Float.toInt(%insert.labelledArgument("minutes")),
14981523
~seconds=Float.toInt(%insert.labelledArgument("seconds")),
14991524
~milliseconds=Float.toInt(%insert.labelledArgument("milliseconds")),
@@ -1592,7 +1617,8 @@ futureTime == Js.Date.getTime(date1)
15921617
@send
15931618
@deprecated({
15941619
reason: "Use `Date.setUTCSecondsMs` instead.",
1595-
migrate: Date.setUTCSecondsMs(
1620+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
1621+
Date.setUTCSecondsMs(
15961622
~seconds=Float.toInt(%insert.labelledArgument("seconds")),
15971623
~milliseconds=Float.toInt(%insert.labelledArgument("milliseconds")),
15981624
),

packages/@rescript/runtime/Js_math.res

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,8 @@ Js.Math.atan2(~x=-0.0, ~y=-5.0, ()) == -.Js.Math._PI /. 2.0
260260
*/
261261
@deprecated({
262262
reason: "Use `Math.atan2` instead.",
263-
migrate: Math.atan2(~y=%insert.labelledArgument("y"), ~x=%insert.labelledArgument("x")),
263+
migrate: @apply.transforms(["dropUnitArgumentsInApply"])
264+
Math.atan2(~y=%insert.labelledArgument("y"), ~x=%insert.labelledArgument("x")),
264265
})
265266
@val
266267
@scope("Math")

0 commit comments

Comments
 (0)