[
https://issues.apache.org/jira/browse/MBUILDCACHE-60?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17742657#comment-17742657
]
ASF GitHub Bot commented on MBUILDCACHE-60:
-------------------------------------------
ferdnyc commented on PR #86:
URL:
https://github.com/apache/maven-build-cache-extension/pull/86#issuecomment-1633517427
I'm with @elharo — I _hate_ merging my own code to a public repo, and will
generally only do it if it's a non-code change or I can definitively state that
merging won't impact other users.
Otherwise, I figure a second pair of eyes is a sensible precaution on almost
**any** code change, and when there are multiple people with commit access a
good general rule is to have someone other than the PR author do the merge. I
try to review others' changes as much as I can, in the hopes they'll return the
favor.
In that spirit, I've pitched in with an inexpert review of the changes here.
> Remove Guava dependency
> -----------------------
>
> Key: MBUILDCACHE-60
> URL: https://issues.apache.org/jira/browse/MBUILDCACHE-60
> Project: Maven Build Cache Extension
> Issue Type: Dependency upgrade
> Reporter: Elliotte Rusty Harold
> Assignee: Elliotte Rusty Harold
> Priority: Minor
> Labels: pull-request-available
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
