Skip to content

Add setCreated() to Meta#2

Open
rhertogh wants to merge 1 commit intotmilos:masterfrom
rhertogh:patch-1
Open

Add setCreated() to Meta#2
rhertogh wants to merge 1 commit intotmilos:masterfrom
rhertogh:patch-1

Conversation

@rhertogh
Copy link
Copy Markdown
Contributor

Implemented setter for Meta::$created

Implemented setter for `Meta::$created`
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.2%) to 69.953% when pulling 7199292 on rhertogh:patch-1 into 34e9cf4 on tmilos:master.

Copy link
Copy Markdown
Owner

@tmilos tmilos left a comment

Choose a reason for hiding this comment

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

👎 Build have failed.

  • php-cs-fixer whitespace

Comment thread src/Model/Meta.php
{
$this->created = $created;
}

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Build have failed. php-cs-fixer is complaining about whitespace in this line

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.

That's strange, I copied the code from setLastModified() and only changed the names.
Could you please provide the exact message from php-cs-fixer?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Click the red link below that says Travis failed, and find the php 7 build where php cs fixer is executed and you'll see the error message in the build log
https://travis-ci.org/tmilos/scim-schema/jobs/612621460

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.

3 participants