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

String: added a change in the string register #3

Merged
merged 1 commit into from Aug 10, 2017

Conversation

Projects
None yet
2 participants
@alexandr-san4ez
Contributor

alexandr-san4ez commented Aug 9, 2017

default

@polyvertex

This comment has been minimized.

Show comment
Hide comment
@polyvertex

polyvertex Aug 10, 2017

Member

Nice! Looks good. Thanks!

Member

polyvertex commented Aug 10, 2017

Nice! Looks good. Thanks!

@polyvertex polyvertex merged commit 367b822 into Keypirinha:master Aug 10, 2017

@alexandr-san4ez alexandr-san4ez deleted the alexandr-san4ez:string-package branch Aug 11, 2017

@polyvertex

This comment has been minimized.

Show comment
Hide comment
@polyvertex

polyvertex Aug 27, 2017

Member

@alexandr-san4ez just in case you are already using this version of the String package, be aware that I've just commited a slightly corrected version that changes the target of the Case Conversion item. It means that you may end up with the old Cases item (your old version) in your history file, resulting in an error if you try to execute it. Best way to circumvent that is to search for it from KP, select it and press Ctrl+Del to avoid any trouble.

Member

polyvertex commented Aug 27, 2017

@alexandr-san4ez just in case you are already using this version of the String package, be aware that I've just commited a slightly corrected version that changes the target of the Case Conversion item. It means that you may end up with the old Cases item (your old version) in your history file, resulting in an error if you try to execute it. Best way to circumvent that is to search for it from KP, select it and press Ctrl+Del to avoid any trouble.

@alexandr-san4ez

This comment has been minimized.

Show comment
Hide comment
@alexandr-san4ez

alexandr-san4ez Aug 27, 2017

Contributor

Thank you for reporting this. I'll take this into consideration.

Contributor

alexandr-san4ez commented Aug 27, 2017

Thank you for reporting this. I'll take this into consideration.

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