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.

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

Upgrade to scrooge 19.3 #130

Merged
merged 3 commits into from
Apr 3, 2019
Merged

Upgrade to scrooge 19.3 #130

merged 3 commits into from
Apr 3, 2019

Conversation

regiskuckaertz
Copy link

No description provided.

@JustinPinner
Copy link
Member

Sorry, but I don't understand the implications of these changes. Of the functionality that's been removed and/or commented out, where is that done now? Is it unnecessary somehow?

@regiskuckaertz
Copy link
Author

The code that is commented out is responsible for generating the JavaScript code; I'll address that in the future as the sbt-thrift plugin is not maintained anymore.

@regiskuckaertz
Copy link
Author

All clear?

Copy link
Member

@JustinPinner JustinPinner left a comment

Choose a reason for hiding this comment

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

👍

@regiskuckaertz regiskuckaertz merged commit 1aff455 into master Apr 3, 2019
@regiskuckaertz regiskuckaertz deleted the rk-scrooge branch April 3, 2019 13:55
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.

None yet

2 participants