Skip to content

Conversation

@Shredder121
Copy link
Contributor

@Shredder121 Shredder121 commented Jul 21, 2016

I noticed something wasn't quite right, will update pull request accordingly as I complete side tasks.

  • You have read the Spring Data contribution guidelines.

    Will do

  • There is a ticket in the bug tracker for the project in our JIRA.

    Not yet

  • You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.

    No formatting changes made

  • You submit test cases (unit or integration tests) that back your changes.

    Will do

  • You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).

    Will do, if still required (the fix isn't exactly creative expression.

  • You provide your full name and an email address registered with your GitHub account. If you’re a first-time submitter, make sure you have completed the Contributor’s License Agreement form.

    You should have my CLA on file for Pivotal, and Spring Projects.

@mp911de
Copy link
Member

mp911de commented Jul 21, 2016

Thanks for the PR. That's an obvious fix, so no tests/author tags/... required.

@mp911de mp911de self-assigned this Jul 21, 2016
mp911de pushed a commit that referenced this pull request Jul 21, 2016
…etWhereClauses(…). Assert.notNull now checks the id parameter in MappingCassandraConverter.getWhereClauses(…). Original pull request: #77.
@mp911de
Copy link
Member

mp911de commented Jul 21, 2016

That's merged with 6e176ba.

@mp911de mp911de closed this Jul 21, 2016
@Shredder121
Copy link
Contributor Author

Thanks, that's indeed what I figured, so decided to hold off on doing the tests/author tags/copyright date/JIRA etc. until I heard something.

No problem, of course!

@Shredder121 Shredder121 deleted the assert-fix branch July 22, 2016 08:05
jxblum added a commit to jxblum/spring-data-cassandra that referenced this pull request Jul 25, 2016
…ion options to CassandraCqlClusterFactoryBean. Original pull request: spring-projects#77.
jxblum added a commit to jxblum/spring-data-cassandra that referenced this pull request Jul 25, 2016
…e to handle extended configuration when the DataStax API changes. Original pull request: spring-projects#77.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants