« first day (434 days earlier)      last day (3404 days later) » 

12:35 AM
@ProgramFOX @KevinBrown Apparently it's because the line is greater than 512 characters long; they don't highlight changes in longer strings.
 
1:14 AM
Ah, that makes sense I guess
 
CI build passed. Ready to pull!
 
!!/pull
 
@hichris123 Pulling latest from master -- CI build passed.
[ SmokeDetector ] SmokeDetector started at rev 8f6ce53 (hichris1234: fake or original passports) (hosted by Undo)
 
these regexes are getting a little out of hand
 
Yeah... I was thinking on how to split them, but I'm not sure.
 
2:06 AM
[ SmokeDetector ] Repeating characters in answer: Newtonsoft JsonConvert with non-standard keys? by tesme on stackoverflow.com
 
[Blaze] answer flagged by Undo: stackoverflow.com/a/28132463
[Blaze] answer flagged by Undo: stackoverflow.com/a/28132367
 
2:39 AM
@Undo Are those being edited by hand or with some tool?
If they aren't, you can always wrap them in parentheses and break them up across multiple lines
 
3:00 AM
@KevinBrown by hand
 
@hichris123 You can break it up by implicitly joining the string across lines with parentheses
24
A: Pythonic way to create a long multi-line string

JesseIf you don't want a multiline string but just have a long single line string, you can use parentheses, just make sure you don't include commas between the string segments, then it will be a tuple. query = ('SELECT action.descr as "action", ' 'role.id as role_id,' 'role.descr ...

 
huh. I almost want to split the regexes up at this point though... a nearly 600 character regex is hard to read.
 
Multiple lines would help, though
 
True.
 
Are there any better ways to organize them? Either in addition to or in replacement of the categorizing by rule.
 
4:07 AM
[ SmokeDetector ] Bad keyword in title: system32.dll virus keeps wiping my porn folder by Flappot on superuser.com
 
 
3 hours later…
7:10 AM
[ SmokeDetector ] Repeating characters in answer: Bluestacks ask Google Play Service eventhough it was installed by Tushar on android.stackexchange.com
 
7:25 AM
CI build passed. Ready to pull!
 
!!/pull
 
@ProgramFOX Pulling latest from master -- CI build passed.
[ SmokeDetector ] SmokeDetector started at rev d33252d (ProgramFOX: Improve escaping methods.) (hosted by Undo)
 
7:38 AM
CI build passed. Ready to pull!
 
!!/pull
 
@ProgramFOX Pulling latest from master -- CI build passed.
[ SmokeDetector ] SmokeDetector started at rev 319d6b8 (ProgramFOX: Add tests for escape-unescape.) (hosted by Undo)
 
just some test case updates
 
 
3 hours later…
10:56 AM
[ SmokeDetector ] URL in title: [stackoverflow.com/questions/28135474/… by Simran kaur on dba.stackexchange.com
 
in Tavern on the Meta on Meta Stack Exchange Chat, 5 mins ago, by Unihedro
^ how did that markdown fail? !cc @Pro @Undo
 
 
1 hour later…
12:49 PM
[ SmokeDetector ] All-caps title: INSERT INTO TOP OF THE DATABASE by Burak Ulu on stackoverflow.com
 
1:14 PM
[ SmokeDetector ] All-caps title: RETURNING MOST FAVOURITE IN SQL by user3572546 on stackoverflow.com
 
 
2 hours later…
2:56 PM
CI build failed, someone (prolly Undo) borked something!
 
eh, I forgot to remove my intentionally borked test file
 
CI build passed. Ready to pull!
CI build passed. Ready to pull!
 
k, now I can run borked test files to check it
 
CI build failed, someone (prolly Undo) borked something!
 
yay
... why does Outlook.com place my Travis emails under "Junk"?
another test running...
 
3:00 PM
CI build passed. Ready to pull!
CI build passed. Ready to pull!
 
oh, right, I forgot that PyCharm automatically uses spaces when I enter a tab.
 
CI build failed, someone (prolly Undo) borked something!
 
okay, it works
 
CI build passed. Ready to pull!
 
all right
 
3:03 PM
CI build passed. Ready to pull!
 
!!/pull
 
[ SmokeDetector ] SmokeDetector started at rev 5d8367f (ProgramFOX: untest) (hosted by Undo)
 
@Undo @hichris123 I added a new feature: srccheck. Circle and Travis now check for some specific rules in the source files:
- Indentation should be spaces, to be consistent with the rest of the project. Using tabs might break it. srccheck returns exit code 1 when it finds a tab.
- Function names should be lower case. We use snake_case for all functions, so srccheck looks whether there are function names with a capitalized letter. If it finds one, it also returns with exit code 1 because that's inconsistent with the other function names.
 
@ProgramFOX You should probably use pytest, just to make things consistent.
 
@hichris123 Right, good idea.
 
3:24 PM
Cool
 
CI build passed. Ready to pull!
 
!!/pull
 
@ProgramFOX Pulling latest from master -- CI build passed.
 
@hichris123 done
 
[ SmokeDetector ] SmokeDetector started at rev c477e61 (ProgramFOX: Move srccheck to pytest) (hosted by Undo)
CI build passed. Commit message contains 'autopull', pulling...
 
3:54 PM
@ProgramFOX Any thoughts on using something like flake8 for doing the basic source checks?
 
4:05 PM
@KevinBrown It certainly looks cool, but we have... a bit too much cases where we "violate" PEP8 (like using a line that's too long). Though it's certainly a good idea to use it when we don't have these cases anymore. Thanks!
 
@ProgramFOX You can always mute/disable those checks and enable them later. I'd need to test it out locally to see how many checks would need to be disabled, but it doesn't look like the code base has completely thrown PEP8 out the window.
 
@KevinBrown Oh, so you can disable a specific kind of error? Cool! I'll check that out.
 
@ProgramFOX Yup, through the configuration file or on the command line.
 
awesome
 
[ SmokeDetector ] Blacklisted user: Axiom of Choice in Mathematics by novice on philosophy.stackexchange.com
 
4:14 PM
Cool, it doesn't even report something about the line length!
 
4:26 PM
erm, wait, it does, I was running the wrong script. facepalm
... wow, it finds many things :O
311 lines of warnings
 
Oh, right. I forgot that it also tries to do some static code analysis.
 
nice
 
You're going to want to disable E501 for the line length check
 
Yeah, I just did -- but still ~100 lines.
 
Might need to put #noqa on the star imports
What are the other warnings you're getting?
Ah, F403 is the code for the star imports if you want to ignore them
 
4:32 PM
E302, E203, E261, E225, E121, E127, E231, E126
and F403 for the stars
 
Perfect time to clean up some code I guess :)
 
If you just want to add /cc @kevin-brown to the issue on GitHub that'd be awesome
 
done
 
4:50 PM
Yeah, ignore E501 @ProgramFOX.
 
yeah
though at places where we can avoid that warning, I'd suggest we actually do avoid it
 
true
Anyway, can PyCharm integrate with it?
 
no idea
I simply run it from the command line
 
@hichris123 I'm pretty sure PyCharm has PEP8 checking built-in, or at least it did when I used it.
 
4:56 PM
Hmm, ok.
I'm using Sublime currently -- no IDE. I probably should start using PyCharm.
 
Sublime definitely has a Flake8 plugin
 
Oh really?
Oh right I never installed Package Control here.
 
phew, I could make ws.py warning-free
 
CI build passed. Commit message contains 'autopull', pulling...
 
Some of those changes are a little annoying to me.
Like github.com/Charcoal-SE/SmokeDetector/commit/… - it's only a few characters longer, and it makes it a little harder to read
 
5:09 PM
k, those can be reverted
 
CI build passed. Commit message contains 'autopull', pulling...
 
Some basic recommendations as far as splltting up lines
 
5:41 PM
!!/pull
 
@ProgramFOX Pulling latest from master -- CI build passed.
 
no idea why it didn't autopull
 
!!/pull
 
@ProgramFOX Pulling latest from master -- CI build passed.
CI build failed, someone (prolly Undo) borked something!
CI build failed, someone (prolly Undo) borked something!
 
that's correct -- I added Flake8 to CI and tested it
 
6:10 PM
CI build passed. Ready to pull!
CI build passed. Ready to pull!
[ SmokeDetector ] SmokeDetector started at rev 2fbd22a (ProgramFOX: untest) (hosted by Undo)
 
Why does it report "CI build passed" twice?
 
@ProgramFOX Travis & CircleCI
 
@hichris123 True, but it used to report only CircleCI.
 
I think it always has, just sometimes the message is swallowed up by the system (ratelimits)
 
Perhaps. But I also never got Travis emails until today.
Perhaps my travis.yml change poked something.
 
6:13 PM
@ProgramFOX Travis only emails when you break it. :P
 
@KevinBrown ^ Flake8 is now in use :)
 
6:30 PM
:)
 
6:51 PM
!!/alive?
@SmokeDetector delete
 
40 mins ago, by ProgramFOX
yay tests!
huh
 
47 secs ago, by SmokeDetector
@ProgramFOX Of course
@hichris123 You... you... mess with my messages! ಠ_ಠ
:P
 
@ProgramFOX Well, you had the closest message. :P
 
4 mins ago, by SmokeDetector
@ProgramFOX Of course
hm, still works
 
7:31 PM
[ SmokeDetector ] All-caps title: LOWER AND UPPER TRIANGULAR MATRIX by jliu on math.stackexchange.com
 
 
2 hours later…
9:05 PM
[Blaze] answer flagged by hichris123: stackoverflow.com/a/28141413
[Blaze] answer flagged by hichris123: stackoverflow.com/a/28141307
 
9:25 PM
-1
A: Need help- basic Java code.(Fibonacci Series)

aGerI don't really understand your question...

srsly?
seriously?
 
 
1 hour later…
10:38 PM
[ SmokeDetector ] URL in title: http://www.dosgamesarchive.com/ trustworthy? by sds537 on gaming.stackexchange.com
 
10:52 PM
[ SmokeDetector ] Offensive answer detected: How to configure a static IP for a specific access point on OS X? by mrtvocich on apple.stackexchange.com
[ SmokeDetector ] Offensive answer detected: Where is Cedric during the quest "Where is Triss Merigold" in The Witcher 2? by LOKO on gaming.stackexchange.com
 
11:24 PM
[ SmokeDetector ] All-caps title: NPN SWITCH WHAT IS WRONG? by 7uc0 on electronics.stackexchange.com
 
11:50 PM
CI build passed. Ready to pull!
 
!!/pull
 
@hichris123 Pulling latest from master -- CI build passed.
[ SmokeDetector ] Offensive answer detected: Who are these women coming at the end of movie Antichrist? by Hector on movies.stackexchange.com
 

« first day (434 days earlier)      last day (3404 days later) »