Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

[GAPRINDASHVILI] Adjusts printwidth to 120, ignores all strings for intl that exceed #446

Conversation

AllenBW
Copy link
Member

@AllenBW AllenBW commented Jun 27, 2018

@AllenBW
Copy link
Member Author

AllenBW commented Jun 28, 2018

thanks @kedark3 for the bz!!!! <3 🙇

@mzazrivec
Copy link
Contributor

Merge anyone? I'd like to rebase #440 on top of this one (thanks Allen).

@AparnaKarve
Copy link
Contributor

@simaishi Can you do the honors here? Thanks.

@simaishi
Copy link
Contributor

@AllenBW @mzazrivec Both this PR and #440 are needed for https://bugzilla.redhat.com/show_bug.cgi?id=1596264 ?

@AllenBW
Copy link
Member Author

AllenBW commented Jun 28, 2018

@simaishi correct! yeah its a catchall translation bz 😋

@mzazrivec
Copy link
Contributor

@simaishi Yup, although I have to rebase #440 before it can be merged.

@AllenBW AllenBW force-pushed the bug/gaprindashvili/adjust-printwidth-for-intl branch 4 times, most recently from 3d3f18c to cdcf324 Compare July 2, 2018 16:46
@simaishi
Copy link
Contributor

@AllenBW @mzazrivec Sorry, I merged the other one first (please mark as WIP in the future for cases like this!)

@AllenBW
Copy link
Member Author

AllenBW commented Jul 11, 2018

😭 ok will get to this in a bit

@AllenBW AllenBW force-pushed the bug/gaprindashvili/adjust-printwidth-for-intl branch from cdcf324 to 39cf990 Compare July 11, 2018 17:48
@AllenBW
Copy link
Member Author

AllenBW commented Jul 11, 2018

@simaishi okie dokie, we're g2g, yah did the right thing thou... this one was gonna cause trouble either way, either everything else backported would break or this one would, much prefer the single place to fix 😏 💟

@simaishi simaishi merged commit c67f803 into ManageIQ:gaprindashvili Jul 11, 2018
@AllenBW AllenBW deleted the bug/gaprindashvili/adjust-printwidth-for-intl branch July 20, 2018 15:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants