Skip to content

Fix spelling: s/user defined/user-defined/#2077

Merged
dlang-bot merged 1 commit intodlang:masterfrom
wilzbach:uda-spelling
Jan 11, 2018
Merged

Fix spelling: s/user defined/user-defined/#2077
dlang-bot merged 1 commit intodlang:masterfrom
wilzbach:uda-spelling

Conversation

@wilzbach
Copy link
Copy Markdown
Contributor

@wilzbach wilzbach commented Jan 11, 2018

See #2003 (comment) for the motivation (it has been requested by @andralex).

@dlang-bot
Copy link
Copy Markdown
Contributor

Thanks for your pull request, @wilzbach!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

spec/traits.dd Outdated

$(P
For more information, see: $(DDSUBLINK spec/attribute, uda, User Defined Attributes)
For more information, see: $(DDSUBLINK spec/attribute, uda, User-defined Attributes)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Probably should capitalize defined here, to be consistent with other places where it's capitalized?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah, it's in a title. I think standard English rules should apply.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

OK. changed.

Copy link
Copy Markdown
Member

@quickfur quickfur left a comment

Choose a reason for hiding this comment

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

LGTM

@dlang-bot dlang-bot merged commit b285392 into dlang:master Jan 11, 2018
@wilzbach wilzbach deleted the uda-spelling branch January 13, 2018 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants