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

Removing no longer used code in the query engine #6382

Merged
merged 1 commit into from
Apr 15, 2016

Conversation

jsternberg
Copy link
Contributor

No description provided.

@corylanou
Copy link
Contributor

+1 with a changelog.

@jsternberg
Copy link
Contributor Author

Where in the changelog should I put dead code removal? Is that a bugfix or a feature?

@corylanou
Copy link
Contributor

I would put it under Bug Fix, as it feels like it should of been removed during a prior commit. But regardless, I don't think it fits under feature.

@jsternberg jsternberg force-pushed the js-influxql-unused-code branch from cb4058c to a37e2db Compare April 14, 2016 20:44
@jsternberg jsternberg force-pushed the js-influxql-unused-code branch from a37e2db to 9c2e430 Compare April 15, 2016 14:22
@jsternberg jsternberg merged commit 6f5c72e into master Apr 15, 2016
@jsternberg jsternberg deleted the js-influxql-unused-code branch April 15, 2016 15:12
@timhallinflux timhallinflux added this to the 0.13.0 milestone Dec 20, 2016
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.

3 participants