the commit rules totally get screwy with links

Question

< Will Jones> the commit rules totally get screwy with links heavy multiplication x body s lines must not be longer than 100 characters body max line length Can you disable this check

Will Jones · Answer

Good to know I ll look into that

rtyler · Answer

I am attempting to fix up the python bindings for arrow 37 but I may need to appeal for help there too I don t spend any time over there stuck out tongue

rtyler · Answer

< Will Jones> Do you have a quick 15 30min to convert this field data type conversion stuff in `python src schema rs` I don t fully understand this code If not I can probably brute force it later tonight after work if my brain still works confused

rtyler · Answer

I made the simple changes

Will Jones · Answer

I can take a look later today

rtyler · Answer

< Will Jones> I got it knocked out on my lunch break laughing pushing changes once the compile finishes and then we should be ready to review

rtyler · Answer

OHNOES what compiled before now fails that everything is updated some of these field mutations have to be handled differently with 37 because of the move from Field to FieldRef in a lot of APIs

rtyler · Answer

bugger

rtyler · Answer

< Will Jones> do you happen to know how `convert to large type` is getting away with the `with data type` without a `mut` modifier

rtyler · Answer

<https docs rs arrow schema 36 0 0 arrow schema struct Field html method with data type|with data type> does take a `mut self` but I m confused by no `mut` coming in from our code here confused

Will Jones · Answer

Not sure I understand the issue it is taking an owned value

Will Jones · Answer

might need to deref and clone the fieldref if you want to get an owned field

rtyler · Answer

ahhhhhhhhhhhh right I ve been doing so much Arc nonsense the last couple days I wasnm t even thinking that with data type was taking ownership

rtyler · Answer

I m so used to Arc and ` amp mut` laughing

Will Jones · Answer

Well this is dumb There doesn t seem to be an easy way to disable it disappointed <https github com conventional changelog commitlint issues 2930>

Will Jones · Answer

actually maybe I can

rtyler · Answer

< Will Jones> looks like we re ready to roll thanks for the assist

Will Jones · Answer

Here we go <https github com delta io delta rs pull 1317>