7f9526bb81
Fix revolve
2025-01-27 08:57:36 +13:00
aafa1a398d
Fix shell
2025-01-27 08:57:36 +13:00
fda7acf0ff
Fix many bad regex find-replaces
...
I wrote a regex find-and-replace which converted `line` calls from
positional to keyword calls. But it was accidentally applied to more
places than it should be, for example, angledLine, xLine and yLine calls.
Fixes this.
2025-01-27 08:57:36 +13:00
bf306e774d
34 failing
2025-01-27 08:57:36 +13:00
2abd516db4
Same as previous
2025-01-27 08:57:36 +13:00
e26d7e741f
Same as previous
2025-01-27 08:57:36 +13:00
91df6b9511
Style fix: skip % when creating new line calls
...
The unlabeled arg is automatically set to % anyway.
2025-01-27 08:57:36 +13:00
3cecb99aa1
Fix: mutateKwArg was appending not overwriting args
2025-01-27 08:57:36 +13:00
e8961d4190
Fix 'remove constraint' on kw arg calls
2025-01-27 08:57:36 +13:00
da3788afba
Fix: make Toolbar components support CallExpressionKw
2025-01-27 08:57:36 +13:00
12ed315244
A snapshot a day keeps the bugs away! 📷 🐛 (OS: namespace-profile-ubuntu-8-cores)
2025-01-27 08:57:33 +13:00
9c35c06e58
Rebasing fixups
...
Signed-off-by: Nick Cameron <nrc@ncameron.org >
2025-01-27 08:56:37 +13:00
a69da088cd
Some line fixes
...
Signed-off-by: Nick Cameron <nrc@ncameron.org >
2025-01-27 08:56:37 +13:00
38e59df85c
Fix some uses of close and lineTo
...
Signed-off-by: Nick Cameron <nrc@ncameron.org >
2025-01-27 08:56:37 +13:00
8bac097743
Fix the 'constrain length' test
2025-01-27 08:56:37 +13:00
09df8a734a
Fix some uses of extrude
...
Signed-off-by: Nick Cameron <nrc@ncameron.org >
2025-01-27 08:56:37 +13:00
da2072e005
Fix drafting lines
2025-01-27 08:56:37 +13:00
cb4a47705c
Fix point-and-click sketch mode to use kwargs
2025-01-27 08:56:37 +13:00
1227b38e12
Update e2e test code with new syntax
2025-01-27 08:56:37 +13:00
8e343a254f
Remove expect from application code
2025-01-27 08:56:37 +13:00
2559309f85
Update snapshots
2025-01-27 08:56:37 +13:00
26f01bce76
fmt
2025-01-27 08:56:37 +13:00
729a34efed
Fixed more e2e tests
2025-01-27 08:56:37 +13:00
5191013db2
Correct sourceRange expectations
2025-01-27 08:56:37 +13:00
d653b852dd
Add back a missing import
2025-01-27 08:56:37 +13:00
f3a2922ba0
Remove .only and fix resulting tests
2025-01-27 08:56:37 +13:00
ce8aeab508
Start fixing playwright tests
2025-01-27 08:56:37 +13:00
ee6014d0cf
Update helix docs
2025-01-27 08:56:37 +13:00
bd329de3e3
Fix the last graph test
2025-01-27 08:56:37 +13:00
5d1df40de5
WIP
2025-01-27 08:56:37 +13:00
21ea1d785a
Support kwargs in addEdgeTreatment
2025-01-27 08:56:37 +13:00
987152e166
Fix some graph tests
2025-01-27 08:56:37 +13:00
6b37bf1809
Start using kwargs branch of kcl-samples
2025-01-27 08:56:37 +13:00
617dcfee34
Fix git merge conflicts from Helix and Cache work
2025-01-27 08:56:37 +13:00
10c446151b
Fix the last test
...
Test was looking for `lineTo` as a substring of the input KCL program.
But there's no more lineTo function, so I changed it to look for
line() with an endAbsolute arg, which is the new equivalent.
Also changed the getConstraintInfo code to look up the lineTo if using
line with endAbsolute.
2025-01-27 08:56:37 +13:00
99866b5f9a
Convert close from positional to kwargs
...
If the close() call has 0 args, or a single unlabeled arg, the parser
interprets it as a CallExpression (positional) not a CallExpressionKw.
But then if a codemod wants to add a tag to it, it tries adding a kwarg
called 'tag', which fails because the CallExpression doesn't need
kwargs inserted into it.
The fix is: change the node from CallExpression to CallExpressionKw, and
update getNodeFromPath to take a 'replacement' arg, so we can replace
the old node with the new node in the AST.
2025-01-27 08:56:37 +13:00
cd022fb087
close is now a kw function
2025-01-27 08:56:37 +13:00
fda4aeb505
Bugfix: don't duplicate tag
2025-01-27 08:56:37 +13:00
d9fe9bafba
Fix another test
2025-01-27 08:56:37 +13:00
9dc06f6b92
Fix another test, allow it to use callExpressionKw
2025-01-27 08:56:37 +13:00
9e778280b3
Fix another test, addTagKw
...
addTag helper was not aware of kw args.
2025-01-27 08:56:37 +13:00
141c38c92b
Convert old test KCL to kwargs
2025-01-27 08:56:37 +13:00
fc7e6993ca
Update sourceRanges
2025-01-27 08:56:37 +13:00
ad8336ef57
Call the KW helper, not just positional
2025-01-27 08:56:37 +13:00
a819d16798
Only 13 failing
2025-01-27 08:56:37 +13:00
1c8cc54c9d
Update test to expect kwargs syntax
2025-01-27 08:56:37 +13:00
70766cef69
Fix tests
...
Assume the line() kwargs calls are being generated in a |>, so they should
not set an unlabeled arg (it will default to % which it was previously setting)
2025-01-27 08:56:37 +13:00
a69ccf960f
Refactor: move some strings into named constants
2025-01-27 08:56:37 +13:00
308b7121fb
Fix more queryAst code that wasn't aware of CallExpressionKw
...
Down to 15 tests failing!!!!!!!!!!
YEAH BABYYYY!!!
2025-01-27 08:56:37 +13:00
6522f1cbe7
Fix another queryAst test
2025-01-27 08:56:37 +13:00
