Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

[Branch repo-nexus-client] #542

Closed
wants to merge 7 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

heroldus commented Sep 14, 2012

Sorry, I want to merge into branch repo-nexus-client

Repository subsystem does not work for group repositories

The code in branch repo-nexus-client does not work for group repositories, because RepositoryGroupResourceResponse can't be casted to RepositoryResourceResponse in JerseyRepositorySupport.
The fix introduces a new abstract method to override the response class.

Extend ArtifactMaven subsystem to support upload artifact

Not finished yet, but good starting point

@peterlynch peterlynch commented on the diff Sep 19, 2012

...ient/core/subsystem/repository/GenericRepository.java
+ * Copyright (c) 2007-2012 Sonatype, Inc.
+ * All rights reserved. Includes the third-party code listed at http://links.sonatype.com/products/nexus/oss/attributions.
+ *
+ * This program and the accompanying materials are made available under the terms of the Eclipse Public License Version 1.0,
+ * which accompanies this distribution and is available at http://www.eclipse.org/legal/epl-v10.html.
+ *
+ * Sonatype Nexus (TM) Professional Version is available from Sonatype, Inc. "Sonatype" and "Sonatype Nexus" are trademarks
+ * of Sonatype, Inc. Apache Maven is a trademark of the Apache Software Foundation. M2eclipse is a trademark of the
+ * Eclipse Foundation. All other trademarks are the property of their respective owners.
+ */
+package org.sonatype.nexus.client.core.subsystem.repository;
+
+import org.sonatype.nexus.rest.model.RepositoryResource;
+
+/**
+ * TODO
Member

peterlynch commented Sep 19, 2012

Don't know enough about the low level stuff to comment on some of this, but generally there are code formatting issues ( use code format defined in https://github.com/sonatype/nexus/blob/master/sonatype-nexus-eclipse-formatter.xml ), documentation issues on interfaces especially ( many TODO ), and appears some test coverage lacking although haven't fully confirmed.

Contributor

heroldus commented Sep 19, 2012

Hi peter,

the code in general is written by Alin and not finished yet. That's why I wanted to merge into branch repo-nexus-client. I justed fixed an issue with group repositories.
Thanks for the eclipse formatter link.

Contributor

adreghiciu commented Oct 31, 2012

I will close this pull request as there is already a new repository client: cc31053

@adreghiciu adreghiciu closed this Oct 31, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment