Skip to content

Rewrite in TypeScript#81

Open
rhys-vdw wants to merge 7 commits into
masterfrom
typescript
Open

Rewrite in TypeScript#81
rhys-vdw wants to merge 7 commits into
masterfrom
typescript

Conversation

@rhys-vdw
Copy link
Copy Markdown
Collaborator

@rhys-vdw rhys-vdw commented Sep 5, 2019

Completely rewrite index.ts in TS.

Comment thread erb_transformer.rb
end
rescue => error
STDERR.puts error
end No newline at end of file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Add line break to the end

Comment thread .npmignore
@@ -0,0 +1 @@
/src No newline at end of file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Add line break to the end

Comment thread .eslintrc.js
ecmaVersion: 2018,
sourceType: 'module',
},
}; No newline at end of file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Add line break to the end

Comment thread erb_transformer.rb
else
puts "#{delimiter}#{handler.new(STDIN.read).result}#{delimiter}"
end
rescue => error
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why is this required in "rewrite to TS"?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Hey, I can't remember now. I probably just fixed this in testing as an attempt to capture error output.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

oh yeah, I think this was to fix #63.

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.

2 participants