Skip to content

Conversation

@vv12131415
Copy link
Contributor

close #2861

I've moved description from property comment to class level annotaion

@vv12131415 vv12131415 changed the title close #2861 move Config::$settings description to class annotations Feb 10, 2020
@jrfnl
Copy link
Contributor

jrfnl commented Feb 11, 2020

Just a note: something weird is going on as Travis isn't building this PR, even though their status page says all systems are operational.

@vv12131415
Copy link
Contributor Author

Anything that I can do with this issue?

@jrfnl

@jrfnl
Copy link
Contributor

jrfnl commented Feb 13, 2020

@vladyslavstartsev You may want to try to force-push the branch again to see if that gets the Travis build running. May just have been a hickup in the system.

@vv12131415 vv12131415 force-pushed the config-settings-docblock branch from 135b57b to 982593d Compare February 14, 2020 21:39
@vv12131415 vv12131415 requested a review from gsherwood February 14, 2020 21:39
@vv12131415 vv12131415 force-pushed the config-settings-docblock branch 2 times, most recently from 40928aa to 6d8c1b6 Compare February 15, 2020 12:10
@vv12131415 vv12131415 force-pushed the config-settings-docblock branch from 6d8c1b6 to 51d4409 Compare February 16, 2020 21:36
@vv12131415
Copy link
Contributor Author

I've managed to be unflagged by Travis team

If this happens again here, here is the link https://travis-ci.community/t/some-pr-are-not-analyzed-with-message-abuse-detected/1191

@jrfnl
Copy link
Contributor

jrfnl commented Feb 16, 2020

I've managed to be unflagged by Travis team

@vladyslavstartsev Glad to hear it ;-)

@gsherwood gsherwood added this to the 3.5.5 milestone Feb 17, 2020
gsherwood added a commit that referenced this pull request Feb 17, 2020
@gsherwood gsherwood merged commit 51d4409 into squizlabs:master Feb 17, 2020
@gsherwood
Copy link
Member

Thanks for doing this.

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.

Make properties on Config class less magic

3 participants