add __lark_meta__ dunder

[ernestoarbitrio]

TBH I don't know how to test it better and write more meaningful test.
Any suggestion is more than welcome

FIXES #9

[erezsh]

It's a good start, but make sure the meta attributes are filled in correctly. line, column, end_line, etc. Doesn't need to be thorough, but just to show it gets written correctly.

[erezsh]

Thanks.

Sorry I forgot to mention, can you also make sure to test it with a Tree instance? (you can just add a rule to the grammar, and check its meta has attributes too)

[erezsh]

It's odd that the tests are passing.. how can that be if I didn't merge the Lark PR yet?

[ernestoarbitrio]

It's odd that the tests are passing.. how can that be if I didn't merge the Lark PR yet?

I think this is not odd since I did not send the meta through the Tree. The token has the line etc... but the Tree still has meta empty. So I wonder how can I exercise a test for this scenario

[ernestoarbitrio]

It's odd that the tests are passing.. how can that be if I didn't merge the Lark PR yet?

maybe i got it ... writing a test

[ernestoarbitrio]

ok now it should fail. FYI on my machine it passes cause I have the parse_tree edited as you PR in lark

[erezsh]

Looks good! I'll merge it after next Lark release.

[erezsh]

@ernestoarbitrio The tests are still failling after releasing 1.1.4, any idea why?

[ernestoarbitrio]

@ernestoarbitrio The tests are still failling after releasing 1.1.4, any idea why?

looking into it

[ernestoarbitrio]

Now it should work

[erezsh]

Great, thank you for contributing!