Skip to content
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

remove fuzzy warnings #898

Merged
merged 1 commit into from Oct 23, 2012
Merged

Conversation

pitr-ch
Copy link
Member

@pitr-ch pitr-ch commented Oct 23, 2012

works with rpm-gem and up-to-date version of FastGettext

works with rpm-gem and up-to-date version of FastGettext
@lzap
Copy link
Contributor

lzap commented Oct 23, 2012

Cooooooooooooooooooool. Now even unit test should be silenced! YAY! ACKnMERGE (AnM :-)

lzap added a commit that referenced this pull request Oct 23, 2012
@lzap lzap merged commit 6f5e1b5 into Katello:master Oct 23, 2012
@xsuchy
Copy link
Contributor

xsuchy commented Oct 29, 2012

Why we needed to silence? Now we can miss other warnings (other than fuzzy strings) and not notice them at all.

@lzap
Copy link
Contributor

lzap commented Oct 30, 2012

Because there were so many messages and Travis, built on javascript, was
totally unusable.

We still have all the warnings in the build phase, anyone can see koji
to see it in full blossom.

LZ

On Mon, Oct 29, 2012 at 09:39:46AM -0700, Miroslav Suchý wrote:

Why we needed to silence? Now we can miss other warnings (other than fuzzy strings) and not notice them at all.


Reply to this email directly or view it on GitHub:
#898 (comment)

Later,

Lukas "lzap" Zapletal
#katello #systemengine

lzap added a commit that referenced this pull request Apr 27, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants