[ 
https://issues.apache.org/jira/browse/OPENJPA-2997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18106249#comment-18106249
 ] 

Maxim Solodovnik commented on OPENJPA-2997:
-------------------------------------------

[~rzo1] Shall this part: {{state the rationale in the PR description so the 
removal is clearly intentional.}} be reflected in CHANGELOG/README or some 
other document?

> Deletion of TestSecurityContext needs rationale in PR
> -----------------------------------------------------
>
>                 Key: OPENJPA-2997
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-2997
>             Project: OpenJPA
>          Issue Type: Sub-task
>          Components: jpa
>    Affects Versions: 4.2.0
>            Reporter: Maxim Solodovnik
>            Priority: Major
>             Fix For: 4.2.0
>
>
> Discussion thread: 
> https://github.com/apache/openjpa/pull/144#discussion_r3683007780
> **(low)** This is the only deleted test file; deletion looks justified 
> (SecurityManager removal on modern JDKs), but please state the rationale in 
> the PR description so the removal is clearly intentional.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to