Skip to content

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

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

2.0.0 #7

wants to merge 15 commits into from

Conversation

wkirby
Copy link
Contributor

@wkirby wkirby commented Jan 29, 2019

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.

import typescript from "rollup-plugin-typescript";
import pkg from "./package.json";

export default [

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";

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";

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";

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";

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').

@wkirby wkirby requested a review from ncallaway January 29, 2019 03:22
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling d546e53 on 2.0.0 into a896540 on master.

@coveralls
Copy link

coveralls commented Jan 29, 2019

Coverage Status

Coverage remained the same at 100.0% when pulling 4e45507 on 2.0.0 into a896540 on master.

@ncallaway
Copy link

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",

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants