Skip to content

noAck -> autoAck #297

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

Merged
merged 3 commits into from
Dec 9, 2016
Merged

noAck -> autoAck #297

merged 3 commits into from
Dec 9, 2016

Conversation

kjnilsson
Copy link
Contributor

Fixes #255

This renames the noAck parameter to autoAck.

Before we release this change we need to update rabbitmq-website and rabbitmq-tutorials with the new name.

@michaelklishin michaelklishin self-assigned this Dec 9, 2016
Copy link
Contributor

@michaelklishin michaelklishin left a comment

Choose a reason for hiding this comment

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

Building from scratch fails with

/Users/antares/Development/RabbitMQ/umbrella.git/deps/rabbitmq_dotnet_client/gensrc/autogenerated-api-0-9-1.cs(3841,13): error CS1061: 'BasicConsume' does not contain a definition for 'm_autoAck' and no extension method 'm_autoAck' accepting a first argument of type 'BasicConsume' could be found (are you missing a using directive or an assembly reference?)
/Users/antares/Development/RabbitMQ/umbrella.git/deps/rabbitmq_dotnet_client/gensrc/autogenerated-api-0-9-1.cs(3853,13): error CS1061: 'BasicGet' does not contain a definition for 'm_autoAck' and no extension method 'm_autoAck' accepting a first argument of type 'BasicGet' could be found (are you missing a using directive or an assembly reference?)

so looks like Apigen needs updating, too (and possibly special casing for this one attribute).

@kjnilsson
Copy link
Contributor Author

@michaelklishin I'll fix that. Must have forgotten to re-run autogen.

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