New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Yaml using object serialization #1128

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@rover12421
Contributor

rover12421 commented Jan 7, 2016

Better operating apktool.yml file (MetaInfo)

rover12421 added some commits Jan 7, 2016

Apktool.yml file (MetaInfo) using object serialization
non-printable character more intuitive, does not affect printable
character view

@rover12421 rover12421 changed the title from Yaml object to Yaml using object serialization Jan 7, 2016

@iBotPeaches

This comment has been minimized.

Show comment
Hide comment
@iBotPeaches

iBotPeaches Jan 7, 2016

Owner

Sweet! Quite an improvement over the previous change, so we don't have to look at encoded strings that make the target file name unreadable.

Theres a few spacing things off, some files modified that don't have anything to do with MetaInfo and some license headers missing, but I can fix those with a quick commit over the top.

A tad bit tied up today and tomorrow, but I'll get to this soon. Thanks again rover

Owner

iBotPeaches commented Jan 7, 2016

Sweet! Quite an improvement over the previous change, so we don't have to look at encoded strings that make the target file name unreadable.

Theres a few spacing things off, some files modified that don't have anything to do with MetaInfo and some license headers missing, but I can fix those with a quick commit over the top.

A tad bit tied up today and tomorrow, but I'll get to this soon. Thanks again rover

@iBotPeaches

This comment has been minimized.

Show comment
Hide comment
@iBotPeaches

iBotPeaches Jan 22, 2016

Owner

Not sure why this didn't auto close - 114af97

Thanks rover!

Owner

iBotPeaches commented Jan 22, 2016

Not sure why this didn't auto close - 114af97

Thanks rover!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment