-
-
Notifications
You must be signed in to change notification settings - Fork 1
2.0.0 #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
import typescript from "rollup-plugin-typescript"; | ||
import pkg from "./package.json"; | ||
|
||
export default [ |
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.
'export' is only available in ES6 (use 'esversion: 6').
import resolve from "rollup-plugin-node-resolve"; | ||
import commonjs from "rollup-plugin-commonjs"; | ||
import typescript from "rollup-plugin-typescript"; | ||
import pkg from "./package.json"; |
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.
'import' is only available in ES6 (use 'esversion: 6').
@@ -0,0 +1,20 @@ | |||
import resolve from "rollup-plugin-node-resolve"; | |||
import commonjs from "rollup-plugin-commonjs"; | |||
import typescript from "rollup-plugin-typescript"; |
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.
'import' is only available in ES6 (use 'esversion: 6').
@@ -0,0 +1,20 @@ | |||
import resolve from "rollup-plugin-node-resolve"; | |||
import commonjs from "rollup-plugin-commonjs"; |
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.
'import' is only available in ES6 (use 'esversion: 6').
@@ -0,0 +1,20 @@ | |||
import resolve from "rollup-plugin-node-resolve"; |
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.
'import' is only available in ES6 (use 'esversion: 6').
One thing to note is that you can add typescript definitions to a project without having to re-write everything. https://www.typescriptlang.org/docs/handbook/declaration-files/introduction.html Not useful now, but probably good to know for the future. |
@@ -1,27 +1,65 @@ | |||
{ | |||
"name": "orcajs", | |||
"version": "1.1.4", | |||
"main": "./dist/orca.js", | |||
"version": "1.1.5", |
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.
Your branch and PR is 2.0.0
Do you want to bump the major version? If there are breaking API changes, we definitely should. If the API has changed and the only definition are typescript definitions, then a minor bump seems OK.
This is a complete overhaul of Orca to be written in Typescript and bundled with Rollup for cjs, esm, and umd formats. It also does a number of small things to simplify the code, normalize naming, and expand spec coverage.