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

fix: exposed-bom module missing when publishing #1818

Merged
merged 1 commit into from
Aug 4, 2023

Conversation

joc-a
Copy link
Collaborator

@joc-a joc-a commented Aug 2, 2023

The exposed-bom module was missing when 0.42.0 was published because its build.gradle.kts file was missing the necessary setup that has credentials and the Space respository URL to publish to. So this setup was copied over from publishing.gradle.kts.

exposed-bom is now already published at the time of writing. A temporary branch was used for the Deploy build config in TeamCity (checked out from this commit 74fba00a77dce1d12cafad722064db2f5aeff84e because it was the last one used when 0.42.0 was released) that includes one more commit with the same changes in this PR, in addition to other changes to prevent re-publishing the other modules and allow publishing only exposed-bom this time.

The changes in this PR are sufficient to include exposed-bom in the next release without any additional work on TeamCity.

@joc-a joc-a requested review from bog-walk and e5l August 2, 2023 10:01
@joc-a joc-a marked this pull request as draft August 2, 2023 10:09
The `exposed-bom` module was missing when `0.42.0` was published because its `build.gradle.kts` file was missing the necessary setup that has credentials and the Space respository URL to publish to. So this setup was copied over from `publishing.gradle.kts`.
@joc-a joc-a force-pushed the joc/include-exposed-bom-in-publishing branch from 6ed4bbb to b34c4a2 Compare August 2, 2023 14:25
@joc-a joc-a changed the title fix: Include exposed-bom module in publishing fix: exposed-bom module missing when publishing Aug 2, 2023
@joc-a joc-a marked this pull request as ready for review August 2, 2023 14:59
@@ -31,13 +31,32 @@ dependencies {
}

publishing {
val version: String by rootProject
Copy link
Member

Choose a reason for hiding this comment

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

rootProject.version is fine as well

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not sure if I'm doing something wrong, but there is a type mismatch.

Screenshot 2023-08-04 at 11 27 42 AM Screenshot 2023-08-04 at 11 29 00 AM

@joc-a joc-a merged commit e6036a3 into main Aug 4, 2023
3 checks passed
@joc-a joc-a deleted the joc/include-exposed-bom-in-publishing branch August 4, 2023 09:30
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.

2 participants