Rename ddoc.d -> ddoc_preprocessor.d#2082
Conversation
|
Thanks for your pull request, @wilzbach! Bugzilla referencesYour PR doesn't reference any Bugzilla issue. If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog. |
|
I find this more confusing than before.
Restructuring should add net value, this is at the very best a lateral step. |
|
FYI: I opened this PR, s.t. #2072 doesn't get blocked. For a bit of history:
|
|
So it seems we added automation that requires us to rename modules in our code. Seems the right place to change things is in the automation, no? |
I'm not sure I can follow. To what "automation" are you referring to? Do you mean the assert->writeln transformation? I don't care how the Ddoc wrapper tool is named as long as the name bike-shedding doesn't block my PRs. Hence, I moved this in a separate PR. |
CyberShadow
left a comment
There was a problem hiding this comment.
Debates about tiny inconsequent changes that take magnitudes more time than the change itself are not a good use of our time.
I liked |
@ZombineDev pointed out that
ddoc.dis a rather uninformative name.