prepare 2.5.0 (#477)

* prepare 2.5.0

Author: AlexHaxe

.travis.yml

@@ -12,6 +12,7 @@ script:
   - haxe build.hxml
   - haxe buildDebug.hxml
   - haxe buildJS.hxml
+  - haxe buildCpp.hxml
   - haxe buildSchema.hxml
   - haxe -D codecov_json buildTest.hxml
   - haxe testJava.hxml
@@ -26,6 +27,7 @@ matrix:
         - haxelib git mcover https://github.com/massiveinteractive/mcover master src
         - haxelib git munit https://github.com/massiveinteractive/MassiveUnit master src
         - haxelib install hxjava
+        - haxelib install hxcpp
         - haxelib install haxeparser 3.3.0
         - haxelib install compiletime 2.6.0
         - haxelib install hxargs 3.0.2
@@ -42,6 +44,7 @@ matrix:
         - haxelib git hxnodejs https://github.com/HaxeFoundation/hxnodejs.git master
         - haxelib git json2object https://github.com/elnabo/json2object
         - haxelib install hxjava
+        - haxelib install hxcpp
         - haxelib install mlib
         - haxelib install mconsole
         - haxelib install compiletime 2.6.0

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## dev branch / next version (2.x.x)
 
+## version 2.5.0 (2019-10-10)
+
 - **Breaking Change** split `OperatorWhitespaceCheck.functionArgPolicy` into `arrowFunctionPolicy`, `oldFunctionTypePolicy` and `newFunctionTypePolicy` [#467](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/467)
 - **Breaking Change** using `--` for long options in cli ([#472](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/472))
 - New check BlockBreakingConditional ([#471](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/471))

buildCpp.hxml

@@ -0,0 +1,10 @@
+buildGlobal.hxml
+# -lib hxcpp-debugger
+# -D HXCPP_DEBUGGER
+
+# -D HXCPP_STACK_TRACE
+# -D HXCPP_STACK_LINE
+
+# -debug
+-main checkstyle.Main
+-cpp out

haxelib.json

@@ -14,12 +14,12 @@
     ],
     "description": "Automated code analysis ideal for projects that want to enforce a coding standard.",
     "contributors": [
-        "adireddy",
         "AlexHaxe",
+        "adireddy",
         "Gama11"
     ],
-    "releasenote": "new checks for doc comments (type, field and style) and redundant @:access/@:allow metas - see CHANGELOG",
-    "version": "2.4.2",
+    "releasenote": "breaking changes: command line options and OperatorWhitespace settings changed, new checks InlineFinal and BlockBreakingConditional, increased Haxe 4 compatibility, some formatter related fixes and a few bugfixes - see CHANGELOG",
+    "version": "2.5.0",
     "url": "https://github.com/HaxeCheckstyle/haxe-checkstyle",
     "dependencies": {}
 }

package.json

@@ -1,6 +1,6 @@
 {
     "name": "haxe-checkstyle",
-    "version": "2.4.2",
+    "version": "2.5.0",
     "description": "Automated code analysis ideal for projects that want to enforce a coding standard.",
     "repository": {
         "type": "git",

test/checkstyle/checks/modifier/InlineFinalCheckTest.hx

@@ -22,13 +22,13 @@ class InlineFinalCheckTest extends CheckTestCase<InlineFinalCheckTests> {
 abstract InlineFinalCheckTests(String) to String {
 	var TEST_INLINE_FINAL = "
 	abstractAndClass Test {
-		public inline final test:String=0;
+		public inline final test:String='0';
 		final function test2() {
 		}
 	}";
 	var TEST_INLINE_VAR = "
 	abstractAndClass Test {
-		inline public var test:String=0;
+		inline public var test:String='0';
 		inline function test2() {
 		}
 	}";

test/checkstyle/checks/modifier/PublicAccessorCheckTest.hx

@@ -6,6 +6,7 @@ class PublicAccessorCheckTest extends CheckTestCase<PublicAccessorCheckTests> {
 	@Test
 	public function testNonAccessors() {
 		assertNoMsg(new PublicAccessorCheck(), NON_ACCESSOR);
+		assertNoMsg(new PublicAccessorCheck(), PRIVATE_ACCESSOR);
 	}
 
 	@Test
@@ -26,6 +27,11 @@ abstract PublicAccessorCheckTests(String) to String {
 		public function _set_test() {}
 		public function _get_test() {}
 	}";
+	var PRIVATE_ACCESSOR = "
+	abstractAndClass Test {
+		private function set_test() {}
+		private function get_test() {}
+	}";
 	var PUBLIC_GETTER = "
 	abstractAndClass Test {
 		public function get_test() {}