-
-
Notifications
You must be signed in to change notification settings - Fork 10
Conversation
cod43156
commented
Oct 8, 2024
Q | A |
---|---|
Documentation | yes/no |
Bugfix | yes/no |
BC Break | yes/no |
New Feature | yes/no |
RFC | yes/no |
QA | yes/no |
@gsteel @alexmerlin |
@arhimede |
This component is still used in some components, like laminas-http. |
@froschdesign Do we need to use Sign-off in commit message mandatorily ? We can amend that commit message if needed. |
Signed-off-by: Raj Mohan <[email protected]>
eaa331b
to
9ed93ad
Compare
@arhimede @froschdesign The DCO issue has been fixed by the author. The only issue left is with the Autocloser, which seems to be failing almost constantly - as seen in the Actions. If the Autocloser check is not mandatory, then this PR can be merged and released. |
@alexmerlin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So we can ignore the Autocloser action in this case.
The repo is anyway in security-only
mode
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @cod43156