aytasato
2017-3-14 14:14:28

@aytasato has joined the channel


leif
2017-3-14 20:15:11

@mflatt Why is scribble/acmart merged into the main repo?


leif
2017-3-14 20:15:28

@mflatt Or rather, why on earth does it use maketitle?


samth
2017-3-14 20:15:46

@leif there was an extensive discussion including discussion of maketitle


leif
2017-3-14 20:15:58

(As opposed to the already existing one on the package server: https://github.com/fare/acmart )


leif
2017-3-14 20:16:14

@samth Okay, link?


samth
2017-3-14 20:16:21

on the pull request


samth
2017-3-14 20:17:07

also, I think “NO!!!” is not a good way to respond to a commit, even if you don’t like the approach


leif
2017-3-14 20:17:51

samth
2017-3-14 20:18:58

“ctrl-f maketitle” on that page works for me


leif
2017-3-14 20:19:03

No shit sam


leif
2017-3-14 20:19:12

But there is no explanation as to why its there.



ben
2017-3-14 20:19:35

I wouldn’t say “extensive discussion”,


ben
2017-3-14 20:19:50

but DVH couldn’t figure out how to insert it automatically. That’s why it’s there


samth
2017-3-14 20:20:01

I meant there was extensive discussion of the PR as a whole, including some discussion of maketitle in particular


leif
2017-3-14 20:20:58

Yes, maketitle was mentioned. But about all I could find is:


leif
2017-3-14 20:21:03
I think that would be a mess. The better thing would be to automatically insert maketitle at the right point, but I couldn't figure out how to do it.

leif
2017-3-14 20:21:24

But I mean, there already is a scribble/acmart implementation that clearly doesn’t need an explicit maketitle.


samth
2017-3-14 20:21:44

well, i suggest submitting a PR with the correct way to do it, then


samth
2017-3-14 20:21:50

I’m sure David would happy


leif
2017-3-14 20:21:57

Fine


leif
2017-3-14 20:22:01

that is acceptable.