Skip to content

typofixes - https://github.com/vlajos/misspell_fixer#489

Merged
jamiehannaford merged 1 commit intorackspace:workingfrom
vlajos:typofixes-vlajos-20141204
Dec 5, 2014
Merged

typofixes - https://github.com/vlajos/misspell_fixer#489
jamiehannaford merged 1 commit intorackspace:workingfrom
vlajos:typofixes-vlajos-20141204

Conversation

@vlajos
Copy link
Copy Markdown
Contributor

@vlajos vlajos commented Dec 4, 2014

No description provided.

Comment thread lib/OpenCloud/Common/Log/Logger.php Outdated
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.

This is a bc break.

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.

This needs to be reverted because, as @GrahamCampbell says, it's a backwards incompatible change that would break semantic versioning.

@vlajos
Copy link
Copy Markdown
Contributor Author

vlajos commented Dec 4, 2014

Should I revert the related parts? Or can we fix this somehow?

Comment thread docs/changelog/1.7.0.md
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.

This can stay

@jamiehannaford
Copy link
Copy Markdown
Contributor

@vlajos Hi Veres, thanks for this - it's very useful! I've gone through and highlighted the areas that need to be reverted, due to semantic versioning. Also, could you rebase your branch against origin/working?

@vlajos vlajos force-pushed the typofixes-vlajos-20141204 branch from f1f264e to 8cfb1a6 Compare December 5, 2014 09:58
@vlajos
Copy link
Copy Markdown
Contributor Author

vlajos commented Dec 5, 2014

Ok. Done.

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 8cfb1a6 on vlajos:typofixes-vlajos-20141204 into * on rackspace:working*.

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