-
Notifications
You must be signed in to change notification settings - Fork 77
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
Ref #3250 Fixes #3251: split repository in ro/rw #116
Ref #3250 Fixes #3251: split repository in ro/rw #116
Conversation
@@ -37,7 +37,7 @@ along with this program. If not, see <http://www.gnu.org/licenses/agpl.html>. | |||
<parent> | |||
<groupId>com.normation</groupId> | |||
<artifactId>parent-pom</artifactId> | |||
<version>2.6.0-SNAPSHOT</version> | |||
<version>2.6.0-scala_2.10-SNAPSHOT</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The pom version should not change, a rebase on should fix that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, clearly, well spotted. I though I changed all of them.
Don't you think that objects named "roXXXrepository" or "woXXXrepository" should be name "xxxReader" or "xxxWriter" ? |
The same questions about the classes "WoXXXRepository" and "RoXXXRepository" being renamed to "XXXWriter" "XXXReader" |
|
||
} | ||
|
||
class RwLDAPDirectiveRepository( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't it be WoLDAPDirectiveRepository ?
Appart from the TW/RO, this is really nice ! I let you answer on or correct the names of the repos |
Thank you ! |
No description provided.