Charcoal HQ

@ArtOfCode It says there's 320 new changes in the repository for MS. Is that right...?

I cloned it mid week-ish

could be

just git pull the lot

Alrighty :)

[ SmokeDetector | MS ] Bad keyword in body, bad keyword in title: Are all the holocaust witnesses jews or nazis? by Liu on skeptics.SE

DavidPostill

10:20 PM

spam? superuser.com/a/1215776/337631

gonna say yes

DavidPostill

!!/report superuser.com/a/1215776/337631

[ SmokeDetector | MS ] Manually reported answer: Why is mega.nz not working with chrome or firefox? by GuestTVP MLP on superuser.com (@HenryWHHackv2.0)

CI on 4420344 succeeded.

The Smokey repo is now using PullApprove. TL;DR: code admins (even those without git rights) can comment "!!/approve" on a PR to merge it. At least one code admin is required to do this to pass status checks.

5

Experimental, for now. We'll see how it goes.

[ SmokeDetector | MS ] Bad keyword in title, bad keyword with email in body, email in body, potentially bad keyword in title: How i got a blank atm card by Eric Gomes on money.SE

tpu- by DavidPostill

10:37 PM

@ArtOfCode Just made my first pull request :O

[ metasmoke ] ci/circleci success on ec37bae: Your tests passed on CircleCI!

^ Awww yeah :)

@Henders It seems like github doesn't recognise the commits as yours, are you sure the email used with git is a email that you have connected to your GH account?

[ SmokeDetector | MS ] Few unique characters in body: delete this please delete this please by NopSled on stackoverflow.com (@quartata)

@Ferrybig Hmm, Maybe I was missing my email

But I cloned it from my own forked version

J F

10:46 PM

@ArtOfCode Does it auto-merge, or does someone with git rights have to merge?

@JF further science required on that one

@Ferrybig Ah, I hadn't added my email into the config :( So it just guessed and used my machine name. Thanks for the heads up

Brock Adams

!!/report stackapps.com/questions/7405/…

[ SmokeDetector | MS ] Manually reported question: Why isn't pprecipitated calcium carbonate (PCC lime) a bigger deal? by First Saturday Lime on stackapps.com

@ArtOfCode Do I need to change what you suggested? I'm not sure how to update my pull request if so :P

10:49 PM

@Henders push another commit to your master branch and the PR will auto-update

You got it :)

@Henders As long as the pull request is not merged: 1. Edit the config 2. Update your last commit git commit --amend --reset-author --no-edit 3. Verify the email is changed using git log 4. Force push to your feature branch git push --force-with-lease, that way its replaced

@Ferrybig Right, I'll do that first. Thanks, that's some nice gitting there!

Nice one, thanks. That's got it :)

[ SmokeDetector | MS ] Bad keyword with email in answer, email in answer: Why is everyone saying how desperately we need to save money "in this economy"? by Okiti Daniel Smith on money.SE

Undo

sd k

Brock Adams

10:54 PM

@SmokeDetector k

tpu- by DavidPostill on Why isn't pprecipitated calcium carbonate (PCC lime) a bigger deal? [MS]

[ metasmoke ] ci/circleci success on 0935115: Your tests passed on CircleCI!

@Henders Email addresses seems to be fine now, the commit is properly attributed

@Ferrybig Thanks for the git tips. I would have had no idea how to do that myself!

What is '--force-with-lease'?

Undo

developer.atlassian.com/blog/2015/04/force-with-lease, looks like

I'd never heard of it either

@ArtOfCode Cool

@Henders No problem, --force-with-lease is a more safer version of --force, the latter can override any commits on the server, while the first can only override a commit if you have at least fetched that commit one time (this may sound a bit confusing, but that is because 'm actually tired right now)

11:01 PM

Hmm okay. I need to improve my git ability.

I made those changes, Art :)

's okay, I hadn't heard of it either

[ metasmoke ] ci/circleci success on 0061d25: Your tests passed on CircleCI!

my strategy for force pushes is "NOBODY DO ANYTHING I'M ABOUT TO DO BAD THINGS"