Squeak compatible: Argument name expected?

[bundgaard]

Loading NeoJSON into Smalltalk

Metacello new 
	baseline: 'NeoJSON'; 
	repository: 'github://svenvc/NeoJSON/repository'; 
	load.

Errored on these lines:

NeoJSON/repository/Neo-JSON-Tests/NeoJSONObjectTests.class.st

Lines 165 to 167 in 11e274d

	paths := json findPaths: [ :_ | true ].
	paths do: [ :each |
	self assert: (json atPath: each) notNil ]

I changed the :_ to :ignored and it went through, is NeoJSON that has to change to be supported on Squeak or do we have to go to Squeak and implement :_?

Or did I misunderstand the error?

Thank you in advance.

[svenvc]

Hi,

NeoJSON was never ported to Squeak. But I believe both NeoJSON and NeoCSV were ported to VAST.

If that change is the only one needed that would be great.

If you could try to load the code with some minimal changes like that and you all the unit tests are green, I would be happy to make that change and you would have contributed to a port.

Please let me know.

Sven

[bundgaard]

Sven, I will try and figure it out. I hope I will return with a PR for you.