-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
README.md needs an update #316
Comments
We are doing heavy refactoring around this part. So things are moving quickly.
…Sent from my iPhone
On 16 Dec 2018, at 13:37, Georgios Andrianakis ***@***.***> wrote:
The Architecture Overview in the README mentions the ResourceProcessor which no longer exists.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Yeah I noticed that looking at the git history :P |
Do we think it's more important to reflect the current state of the code, or the final architecture? If the former, we could spend a lot of time updating outdated docs. OTOH the latter won't be totally accurate (yet). |
Maybe it should be an overview of the final architecture with notes about things that are temporarily different or not implemented yet. |
Maybe we should try to be more concise and avoid pointing to actual class names. Having a general overview should be enough. And maybe point to the root of a very simple extension that we could give as an example. Moving that to |
Most of this is in outher guides now, and does not make sense to be in the README
Fixes #316 remove a lot of out of date info from the readme
The
Architecture Overview
in the README mentions theResourceProcessor
which no longer exists.The text was updated successfully, but these errors were encountered: