Closed ballsteve closed 3 months ago
Hi Steve,
I'm reviewing this at the moment, I gotta say I really like how Smite is working out. :)
Hi Steve,
I've made some small adjustments to fix those parser failing tests, mind taking a quick look? Can merge then.
Yes, the smite module is looking much better. It's nice how code evolves over time.
The changes to the XML parser look good. However, there are now a few errors coming up in other tests. I'll fix those and let you know when the changes are committed.
I've fixed the errors in the document tests. Everything compiles now, and most tests pass. However, there are a few document tests that are failing because they run for more than 60 seconds. Not sure why this is only occurring in document tests, and not unit or integration tests.
I'm going to press ahead with the merge, and re-check these tests in the dev branch.
These changes look good in any case.
Thanks for all your work on this! :)
Just ran them there, the docs pass on my machine but admittedly its a fairly beefy gamer laptop.
There's a lot of warnings being generated by the validator module; particularly it doesn't like the way you're naming functions. Eg:
warning: function oneOrMore
should have a snake case name
--> src/validators/relaxng/derive.rs:468:4
|
468 | fn oneOrMore(pat: Pattern) -> Pattern {
| ^^^^^^^^^ help: convert the identifier to snake case: one_or_more
Although it doesn't bother me too much, if rustc isn't happy then we'd better fix it ;-)
I'll get it fixed, I tend to ignore most warnings until a final release, bad habit I know.
I'm hoping to have more work on it done in the next few days but until I have a decent number of tests passing I'll probably hide it from being publically available, will let you know. On 6 Jun 2024, 05:43 +0100, Steve Ball @.***>, wrote:
There's a lot of warnings being generated by the validator module; particularly it doesn't like the way you're naming functions. Eg: warning: function oneOrMore should have a snake case name --> src/validators/relaxng/derive.rs:468:4 | 468 | fn oneOrMore(pat: Pattern) -> Pattern { | ^^^^^^^^^ help: convert the identifier to snake case: one_or_more Although it doesn't bother me too much, if rustc isn't happy then we'd better fix it ;-) — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because your review was requested.Message ID: @.***>
This PR includes a bunch of features to get us closer to the 1.0 release:
Also a number of cleanups and codebase improvements: