-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Improve Quick start guide for Developers documentation #3411
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@Mishrasubha Can you provide more details about the issue? |
Okay thanks for pointing out the issue. We will fix it. |
@misaochan This is another instance where it would have been great if the volunteers would have been able to help us update the documentation. Given the tight control we want for our docs, we can probably consider some workaround for letting volunteers contribute to wiki documentation. One approach is described here where people maintain a separate fork for the wiki. https://stackoverflow.com/questions/10642928/how-to-pull-request-a-wiki-page-on-github |
@maskaravivek I agree. It's a pity GitHub does not have that option, at the very least they should be able to submit pull requests to it! I actually wonder if we can/should migrate to MD files instead. That way they can be maintained with pull requests like the rest of the repo. The solutions mentioned in your SO thread would have a higher maintenance overhead IMO, especially if many small changes are made. |
I've fixed the docs, just removed the bit about Instant Run now the new android gradle plugin fixes the underlying issue (see #3443) I've been thinking about the wiki, and actually think the best idea would be a mix of the two: have a seperate repo, where we store MD files. I think having it in the main repo doesn't make too much sense (as at the moment changes are quite heavy, as Travis will build an android build for every small change) - plus it means we can be less tight with reviews etc. before merging as it's docs only. If people are happy with that propsal I'm happy to create it and migrate the existing wiki over (shouldn't be too hard, as under the hood it's already a git repo). |
I am happy with this proposal. Let us wait for @misaochan's opinion. |
I'm happy with this as well. We can update our readme.md to point users to the documentation repo, too. :) Thanks @domdomegg ! |
Oh, and don't forget to link to it from the then main page of the wiki for better reach! |
I've moved the wiki to commons-app/documentation - let me know what you think. Always happy to change it with feedback, and the easiest time to do that is now :) If everyone's happy with how that seems (at least for now) I'll update the links from this repo's README. I'll try to go through and fix the non-big-structural issues later (e.g. some articles lacking titles, inconsistent title sizes, outdated docs) |
Thank you for your work @domdomegg ! The new repo looks great IMO. :) |
@domdomegg The repo is really good and well organized. I love the way you've used the emojis. It gives it a really nice touch. 🙂 Great job ❣️ Just a few questions:
I'll be glad to send a PR (which is the whole point of introducing the repo!) that makes the changes related to the suggestions above. Just wanted to get some inputs/opinions before I do that. 🙂 |
No, just that GitHub doesn't store wiki page's titles in the markdown files you export them as - I'm planning on writing a script to add all the titles soon. EDIT: done
Yeah that would be good - just make sure to update the links :)
Feel free to tag me for a review :) |
Thanks a lot @domdomegg :) |
Instant run is missing in Android Studio 3.5 and is replaced with Apply Changes in Run. So, the documentation needs to be updated.
The text was updated successfully, but these errors were encountered: