Skip to content
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

Skip building Yospace connector on JitPack #22

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

MattiasBuelens
Copy link
Contributor

JitPack doesn't have the necessary credentials for the Yospace Maven repository, so it fails to build the project. From the build log:

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:checkDebugAarMetadata'.
> Could not resolve all files for configuration ':app:debugRuntimeClasspath'.
   > Could not resolve com.yospace:admanagement-sdk:3.6.7.
     Required by:
         project :app
      > Could not resolve com.yospace:admanagement-sdk:3.6.7.
         > Could not get resource 'https://yospacerepo.jfrog.io/yospacerepo/android-sdk/com/yospace/admanagement-sdk/3.6.7/admanagement-sdk-3.6.7.pom'.
            > Username must not be null!

Fix it by skipping the Yospace connector altogether on JitPack, so the other connectors can still be built and published. We will only publish the Yospace connector to maven.theoplayer.com.

@MattiasBuelens MattiasBuelens marked this pull request as ready for review June 19, 2024 10:11
@MattiasBuelens MattiasBuelens force-pushed the skip-yospace-on-jitpack branch from c2c6703 to 2c8ac44 Compare June 19, 2024 11:15
@MattiasBuelens MattiasBuelens merged commit 0b5d9e1 into master Jun 19, 2024
2 checks passed
@MattiasBuelens MattiasBuelens deleted the skip-yospace-on-jitpack branch June 19, 2024 11:17
@MattiasBuelens MattiasBuelens added the 🔌 connector: yospace Affects the Yospace connector label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔌 connector: yospace Affects the Yospace connector
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants