__Feature to document__
Move voting-app repo https://github.com/webpack-contrib/voting-app inside that repo
__Author(s)__
Don't known
__Additional information__
Pros:
[ ] I am willing to work on this issue and submit a pull request.
inside that repo
Which repo?
inside that repo
Which repo?
I guess he meant in this repo
If that is the case, why?
This repo should only host documentation, any external app, plugin, etc should be a dependency.
Initial discussion: https://github.com/webpack/webpack.js.org/issues/301
Initial implementation: https://github.com/webpack/webpack.js.org/pull/1717
@montogeek
Pros:
It is not high priority
I'll pick this up, in case we all agree with the change 👍
I dont think this is necessary, its already easy to update: #3617
@EugeneHlushko @montogeek we want to move all repos from webpack-contrib in long term (it is hard to maintance, a lot of people have access and potential do dangerous actions and etc), some of repos will be migrated to monorepo webpack/webpack-official, some of repos will be deprecated, some of them move out from official support, so we start it process, we need move voiting-app to here or maybe webpack/voting-app, I prefer first
we dont want any technical complexity in documentation repository, all modules that are abstractable should live in separate repositories. cc @skipjack
What about move it to webpack/voting-app?
Definitely, that is better :)
On Tue 17. Mar 2020 at 14:33, Evilebot Tnawi notifications@github.com
wrote:
What about move it to webpack/voting-app?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/webpack/webpack.js.org/issues/3614#issuecomment-600073745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHUXXOWWAUK6BTTOW5SHGLRH533RANCNFSM4LF5GFJQ
.>
Fernando Montoya
montogeek.com
@montogeek @EugeneHlushko moved to webpack org https://github.com/webpack/voting-app, need update https://github.com/webpack/webpack.js.org/blob/master/package.json#L170
I'll do it 👍
@anshumanv dont forget to do it via yarn on node 10, ty in advance!
its merged, ty all
Most helpful comment
I dont think this is necessary, its already easy to update: #3617