Skip to content
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

Rewording of documentation #2

Merged
merged 2 commits into from
Sep 30, 2019
Merged

Conversation

jraut
Copy link

@jraut jraut commented Oct 27, 2018

Hello,

I went through the documentation and found these minor details. The word editor was being replaced by the word editor in the example case under Demand-driven change propagation. This was a source of confusion for me since Editor is an role defined earlier in the docs.

Decided to submit a pull request - happy Hactober!

@XVilka
Copy link
Contributor

XVilka commented Sep 29, 2019

Ping? @matthewhammer @kyleheadley

@matthewhammer
Copy link
Member

Thank you @jraut and @XVilka!

My GH notifications were not set up correctly in the past, and (sadly) this project has been neglected this past year. I missed this PR until now. Thanks for the help; I appreciate it.

@matthewhammer matthewhammer merged commit e0ce530 into Adapton:master Sep 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants