Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fixed print issue #36

Merged
merged 15 commits into from
May 6, 2017
Merged

Fixed print issue #36

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
3ff424b
toTex function
aliang8 Apr 30, 2017
9eaf0ba
fixed toTex function
aliang8 Apr 30, 2017
b1ea3d3
forgot a piece
aliang8 Apr 30, 2017
2ec76c6
fixed
aliang8 Apr 30, 2017
1583865
fixed
aliang8 Apr 30, 2017
5102e25
times
aliang8 Apr 30, 2017
d144dc5
equations case
aliang8 Apr 30, 2017
1ff204a
fixed print
aliang8 May 5, 2017
1e728ab
rebased
aliang8 May 5, 2017
b460f6f
linter and print
aliang8 May 6, 2017
142ab42
fixed
aliang8 May 6, 2017
348a9b1
removed script file
aliang8 May 6, 2017
0714ca1
added eslintrc
aliang8 May 6, 2017
419714c
reverted package.json
aliang8 May 6, 2017
f8fb161
oops
aliang8 May 6, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 1 addition & 4 deletions lib/print.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,11 +64,8 @@ const printApply = (node, parent) => {
} else if (op === 'pow') {
const exponent = node.args[1];
let bool = false;
// check if exponent has multiple terms and if the exponent
// contains a div/mul operation
// e.g x^((x + 1)/(2 + 2))
if (exponent.args){
if (exponent.op == 'div' || exponent.op == 'mul'){
if (isMul(exponent) || isDiv(exponent)){
bool = true;
}
}
Expand Down
2 changes: 2 additions & 0 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@
"babel-plugin-transform-object-rest-spread": "^6.22.0",
"babel-preset-es2015": "^6.22.0",
"eslint": "^3.15.0",
"eslint-config-google": "^0.7.0",
"eslint-plugin-sort-requires": "^2.1.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please leave this out. I'm open to changing lint set up for this repo but we should discuss it first and do it in a separate PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup, fixed!

"eslint-plugin-flowtype": "^2.30.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't have been removed either.

"flow-bin": "^0.38.0",
"mocha": "^3.2.0",
Expand Down