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

Misc. typo fixes #265

Merged

Conversation

lorcan
Copy link
Contributor

@lorcan lorcan commented Dec 12, 2016

Came across a couple of typos in the Readme and decided to run a spell check. Hope you don't mind and I'm not being too pedantic ;-)

@lorcan lorcan force-pushed the lorcan/tiny-typo-fixes branch from 96d1782 to 3eafc22 Compare December 12, 2016 11:42
@@ -189,7 +189,7 @@ class ModelDefinition {
fixture[attribute] = fixture[attribute].call(this, fixture);
}

addObjectAtribute(fixture, attribute) {
addObjectAttribute(fixture, attribute) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only change I'm worried about - is there any possible callers of this function apart from earlier in this file (L170)?

@danielspaniel danielspaniel merged commit 2a8e74f into adopted-ember-addons:master Dec 12, 2016
@danielspaniel
Copy link
Collaborator

No at all .. pedantic is good in these cases, because it makes everything more sharp. Especially in documentation, things should be as close to perfect as possible. Thanks for the efforts.

@lorcan lorcan deleted the lorcan/tiny-typo-fixes branch December 12, 2016 18:59
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.

2 participants