Matmaus / LnkParse3

Windows Shortcut file (LNK) parser
MIT License
63 stars 13 forks source link

Merge python3 #8

Closed ernix closed 3 years ago

ernix commented 3 years ago

Cover ca93a61 while resolving merge conflict.

Matmaus commented 3 years ago

Isn't commit 81346e1 the only one needed? I think we do not need others. It seems to be a mess of commits changing only a simple thing to me.

ernix commented 3 years ago

@Matmaus Yes, it is. We can just cherry-pick 81346e1 if you want. Do you want to keep tree diagram as simple as possible?

Matmaus commented 3 years ago

@Matmaus Yes, it is. We can just cherry-pick 81346e1 if you want. Do you want to keep tree diagram as simple as possible?

Yes, I think the simpler is better, and in this case, it can be much simpler.

ernix commented 3 years ago

@Matmaus Got it, wait a sec.