Skip to content

Conversation

@Trott
Copy link
Member

@TrottTrott commented Oct 9, 2015

common does not need util properties anymore. Remove them.

common does not need util properties anymore. Remove them.
@TrottTrott added the test Issues and PRs related to the tests. label Oct 9, 2015
@Trott
Copy link
MemberAuthor

Trott commented Oct 9, 2015

@Fishrock123
Copy link
Contributor

LGTM if CI is happy

@brendanashworth
Copy link
Contributor

LGTM too.

@thefourtheye
Copy link
Contributor

LGTM

@Trott
Copy link
MemberAuthor

Landed in 3202456

@TrottTrott closed this Oct 12, 2015
Trott added a commit that referenced this pull request Oct 12, 2015
common does not need util properties anymore. Remove them. PR-URL: #3304 Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Brendan Ashworth <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Trott added a commit that referenced this pull request Oct 28, 2015
common does not need util properties anymore. Remove them. PR-URL: #3304 Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Brendan Ashworth <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]>
@jasnell
Copy link
Member

Landed in v4.x-staging in 4c411ae

Trott added a commit that referenced this pull request Oct 29, 2015
common does not need util properties anymore. Remove them. PR-URL: #3304 Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Brendan Ashworth <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]>
@TrottTrott deleted the goodbye-util branch January 13, 2022 22:29
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Trott@Fishrock123@brendanashworth@thefourtheye@jasnell