BeiBaiNian opened a new pull request, #50:
URL: https://github.com/apache/casbin-jcasbin-java-cli/pull/50
# Why
casbin-jcasbin-java-cli is an Apache Incubator project; source files should
carry the Apache License 2.0 header. Only 2 of the 11 java files
(DependencyHandler/VersionUtil) had the header; the other 9 were missing. This
PR adds the header to all of them, aligned with the same-language sibling repo
casbin-jcasbin-jdbc-adapter (6/6 files have headers).
# What changed
- Added the standard 14-line header (byte-identical to the 2 existing
headers in this repo) to 9 java files:
- src/main/java/org/casbin/Client.java, CommandExecutor.java,
NewEnforcer.java
- src/main/java/org/casbin/generate/CustomClassLoader.java,
DynamicClassGenerator.java, JavaSourceFromString.java
- src/main/java/org/casbin/resp/ResponseBody.java,
src/main/java/org/casbin/util/Util.java
- src/test/java/org/casbin/ClientTest.java
- Header format: // Copyright 2024 The casbin Authors. All Rights Reserved.
(year = repo's first commit year 2024), identical to the existing headers
- A blank line separates the header block from the package declaration (the
mentor's feedback on typeorm PR #86)
- CRLF line endings preserved everywhere
# Verification
- Local `mvn -B package` passes (BUILD SUCCESS), Tests run: 11, Failures: 0
- git diff confirms only the headers were added (+14 lines per file), no
source changes
- file command confirms CRLF line endings preserved
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
