Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[eclipse-pmc] Change #119156

Dear PMC,

I reviewed and merged today change#119156 (Bug#532273) and made 2 important mistakes:
- The patch adds an API method. In my initial review I commented that PMC approval is needed and thought Conrad would request this then, but it did not happen. However, he addressed the other comments and submitted a change that looks good to me.
- It is M7 quiet week, I did not have this in mind. I thought it was time to review last changes for M7.

Apologies for those mistakes. How to proceed now? Can this change be approved or shall it be reverted?

Conrad covered his change by additional tests in DifferentRealmsBindingTest.

Kind regards,
~Karsten

Back to the top