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

Corrected file path in README.md #348

Merged
merged 2 commits into from
Jan 22, 2018
Merged

Conversation

olivertappin
Copy link
Contributor

No description provided.

@herzinger
Copy link
Contributor

Thanks for the contribution, and for finding the mistake, but there is a long running issue for this type of small correction (#21). I think the best would be to point this to that issue instead,

CHANGELOG.md Outdated
@@ -15,6 +15,7 @@
### Fixes
- Fix `o-layout--stretch` in conjunction with `o-layout--center`, `o-layout--right` and `o-layout--left`. [[#293](https://github.com/inuitcss/inuitcss/issues/293)]
- Update to Sass’ `@else if` syntax according to current spec. [[#331](https://github.com/inuitcss/inuitcss/issues/331)]
- Corrected filename for _objects.table.scss in README.md example. [[#347](https://github.com/inuitcss/inuitcss/issues/347)]
Copy link
Member

@csshugs csshugs Jan 21, 2018

Choose a reason for hiding this comment

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

Please use imperative instead of past tense:

- Correct filename [...]

Copy link
Member

@csshugs csshugs left a comment

Choose a reason for hiding this comment

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

@olivertappin
Copy link
Contributor Author

Comment addressed @csshugs. And @herzinger that's fine. Does this make this PR invalid, or is your reference to that issue enough to get this through?

Copy link
Member

@csshugs csshugs left a comment

Choose a reason for hiding this comment

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

@olivertappin No, it's fine, thank you for the change. Let's not be too pedantic about it.

One small thing for the future: Try to stick to our commit message conventions, so we get an even better paper trail when logging commits. But as I said, let's not be too pedantic about.

Thanks again for pointing out and fixing this error!

@csshugs csshugs merged commit e2d1c33 into inuitcss:develop Jan 22, 2018
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.

None yet

3 participants