Update package name by kirek007 · Pull Request #220 · operator-framework/java-operator-sdk · GitHub
Skip to content

Update package name - #220

Merged
kirek007 merged 4 commits into
masterfrom
update-package-name
Nov 4, 2020
Merged

kirek007 merged 4 commits into
masterfrom
update-package-name

Conversation

@kirek007

@kirek007 kirek007 commented Nov 2, 2020

Copy link
Copy Markdown
Contributor

This PR consist changes related to renaming package and group for code and artifacts from com.github.containersolutions to io.javaoperatorsdk.

Changes includes:
✔ Java source code
✔ Readme file
✔ Maven configuration files
✔ Manifest files
✔ Site files

@kirek007
kirek007 requested a review from adam-sandor November 2, 2020 15:04

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This files consist undesired changes related to auto refactoring code using IDEA, but it looks good.

@@ -38,7 +37,7 @@ public void configMapGetsCreatedForTestCustomResource() {
integrationTestSupport.teardownIfSuccess(() -> {
TestCustomResource resource = testCustomResource();

integrationTestSupport.getCrOperations().inNamespace(TEST_NAMESPACE).create(resource);

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This files consist undesired changes related to auto refactoring code using IDEA, but it looks good.

import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertTrue;

class ControllerUtilsTest {

@Test
public void returnsValuesFromControllerAnnotationFinalizer() {
assertEquals(DEFAULT_FINALIZER, ControllerUtils.getDefaultFinalizer(new TestCustomResourceController(null)));

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This files consist undesired changes related to auto refactoring code using IDEA, but it looks good.

@@ -20,13 +18,13 @@ public void delayIsMultipliedEveryNextDelayCall() {
RetryExecution retryExecution = getDefaultRetryExecution();

Optional<Long> res = callNextDelayNTimes(retryExecution, 2);
assertThat(res.get()).isEqualTo(DEFAULT_INITIAL_INTERVAL);

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This files consist undesired changes related to auto refactoring code using IDEA, but it looks good.

@csviri

csviri commented Nov 2, 2020 •

Copy link
Copy Markdown
Collaborator

@adam-sandor this won't break the release to maven central? Aren't we registered there to some package name?
Also I think "github" needs to be in the package @kirek007

@adam-sandor

Copy link
Copy Markdown
Collaborator

@adam-sandor this won't break the release to maven central? Aren't we registered there to some package name?
Also I think "github" needs to be in the package @kirek007

Yes that is right. There were two options to register ourselves in MC the one that looked simpler was Github. I think for the other you actually have to own the domain (but I don't really remember this). We will definitely have to redo the MC registration process.

@kirek007

kirek007 commented Nov 2, 2020 •

Copy link
Copy Markdown
Contributor Author

@kirek007
kirek007 merged commit f985219 into master Nov 4, 2020
@csviri
csviri deleted the update-package-name branch November 4, 2020 13:03
froque added a commit to froque/oga-maven-plugin that referenced this pull request Dec 12, 2024
> This PR consist changes related to renaming package and group for code and artifacts from com.github.containersolutions to io.javaoperatorsdk.

operator-framework/java-operator-sdk#220

> This release includes new package name and artifact group id

https://github.com/operator-framework/java-operator-sdk/releases/tag/v1.3.0
camerodev added a commit to camerodev/oga-maven-plugin that referenced this pull request Apr 6, 2025
> This PR consist changes related to renaming package and group for code and artifacts from com.github.containersolutions to io.javaoperatorsdk.

operator-framework/java-operator-sdk#220

> This release includes new package name and artifact group id

https://github.com/operator-framework/java-operator-sdk/releases/tag/v1.3.0
rosen1218627 added a commit to rosen1218627/oga-maven-plugin that referenced this pull request Sep 10, 2025
> This PR consist changes related to renaming package and group for code and artifacts from com.github.containersolutions to io.javaoperatorsdk.

operator-framework/java-operator-sdk#220

> This release includes new package name and artifact group id

https://github.com/operator-framework/java-operator-sdk/releases/tag/v1.3.0
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