Add clarification for deployment with submodules

[ctjlewis]

Add clarification for deployment with submodules

[lindsaylevine]

hey @ctjlewis! thanks so much for opening this issue and the accompanying PR!! this is an interesting use case i personally haven't encountered before but a very valid and not uncommon one, of course. there are a few things that come to mind. with the Next on Netlify initiative here at netlify, we will eventually be moving away from this repo. i think it doesn't hurt at all to merge your PR (thanks for the accessibility changes as well!!) with the submodule details; that said, i think it may/will be more appropriate in netlify's docs whenever we get to a point of having a thorough breakdown of the framework in our docs (vs scattered between here and the plugin repo and stuff). that's just me kindof sharing my line of thinking; doesn't really change things for you :). it'll just be on me to make sure this bit gets transferred over to wherever we'll be housing all the next stuff in our docs. lmk if you have any other thoughts! and feel free to tag me in the PR when it's no longer a draft!

[lindsaylevine]

totally with you! is your PR still a draft or is it ready to be reviewed/merged? :D @ctjlewis

[ctjlewis]

I had the same thoughts - this isn't really a Next on Netlify thing, it's really a Deploy footnote in general.

Ideally, like you said, that note would go in the Deploy docs, and there would just be a tiny footnote in the section we have now like "please see this note regarding private submodules," which is much less intrusive on existing docs as well.

I just sent PRs in for all the bumps I hit trying to get netlify dev to work, and this was the final thing that caused the CLI to throw! Not trying to spam pointless changes, hope it's not a bother.

[ctjlewis]

It's probably good to review/merge @lindsaylevine! I will update.