Cleanup Flow typings

* Add explicit Flow typings alongside Typescript typings
* Changed test file suffix
* Changed test file location
* Changed package.json script names for type checking
* Changed Flow config to actually work
* Changed CircleCI config
* Fixed Flow declaration to provide correct and stricter typings
* Removed `lib/` from package.json files
* Removed a bunch of unused Flow types

.flowconfig

@@ -1,12 +1,13 @@
 [ignore]
 
 [include]
-declarations/
 
 [libs]
+declarations/
 
 [options]
 esproposal.class_instance_fields=enable
 esproposal.class_static_fields=enable
 suppress_comment=.*\\$FlowFixMe
 suppress_comment=.*\\$FlowIssue
+module.name_mapper='^moment-range$' -> '<PROJECT_ROOT>/lib/moment-range'

.npmignore

@@ -1,6 +1,7 @@
 declarations/
 flow-typed/
-lib/moment-range_test.js
+lib/tests/*
+*.test.js
 typing-tests/
 package-lock.json
 karma.conf.js

CHANGELOG.md

@@ -29,6 +29,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 * Added note about supporting older browsers with links to polyfills to the README
 * Added moment extension `rangeFromISOString`, changed name from `parseZoneRange`
 * Added `snapTo()` method
+* Added explicit Flow typing tests
 
 ### Changed
 
@@ -40,10 +41,26 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 * Changed `prepublish`, `preversion`, `version` scripts to support typescript definitions
 * Changed CircleCI config to also run typescript tests
 * Changed `parseZoneRange` to `rangeFromISOString` to follow naming conventions. Deprecated `parseZoneRange`.
+* Changed typing tests to be grouped by type-checker (flow, typescript)
+* Changed test suffix (`_test.js` → `.test.js`)
+* Changed test file location (`./lib/` → `./lib/tests/`)
+* Changed package json script names:
+    - `flow` → `check:flow`
+    - `typescript-test` → `check:typescript`
+* Changed CircleCI to use Node 8.2.0
+* Changed CircleCI to use Yarn
+* Changed Flow config to find correct declarations
+* Changed location of Flow declaration (`./declarations/` → `./lib/`)
 
 ### Fixed
 
 * Fixed `intersect` not creating a new DateRange instance in all cases
+* Fixed Flow declaration to provide correct and stricter typings
+
+### Removed
+
+* Removed `lib/` from package.json files
+* Removed a bunch of unused Flow types
 
 ## [3.1.1]
 ### Fixed

README.md

@@ -722,6 +722,7 @@ yarn install
 Do all the things!
 
 ``` sh
+yarn run check
 yarn run test
 yarn run lint
 ```

circle.yml

@@ -1,11 +1,13 @@
 machine:
   node:
-    version: 6.1.0
+    version: 8.2.0
 dependencies:
-  pre:
-    - npm install --global flow-bin
+  override:
+    - yarn
 test:
+  override:
+    - yarn test
   pre:
-    - npm run lint
-    - npm run flow
-    - npm run typescript-test
+    - yarn lint
+    - yarn check:flow
+    - yarn check:typescript

declarations/moment-range.js.flow

@@ -0,0 +1 @@
+../lib/moment-range.js.flow

declarations/moment.js.flow

@@ -235,12 +235,12 @@ declare class moment$Moment {
   fn: Object;
   static range(interval: string): DateRange;
   static range(start: Date, end: Date): DateRange;
-  static range(start: Moment, end: Moment): DateRange;
+  static range(start: moment$Moment, end: moment$Moment): DateRange;
   static isRange(range: any): bool;
   range(interval: string): DateRange;
   range(start: Date, end: Date): DateRange;
-  range(start: Moment, end: Moment): DateRange;
-  rangeFromInterval(interval: string, count?: number, date?: Date|Moment|string): DateRange;
+  range(start: moment$Moment, end: moment$Moment): DateRange;
+  rangeFromInterval(interval: string, count?: number, date?: Date|moment$Moment|string): DateRange;
   rangeFromISOString(isoTimeInterval: string): DateRange;
   within(range: DateRange): bool;