Skip to content

chore(build): prepare for releases#78

Merged
wmontwe merged 15 commits into
thunderbird:mainfrom
wmontwe:chore-build-prepare-for-releases
Jun 25, 2026
Merged

chore(build): prepare for releases#78
wmontwe merged 15 commits into
thunderbird:mainfrom
wmontwe:chore-build-prepare-for-releases

Conversation

@wmontwe

@wmontwe wmontwe commented Jun 19, 2026

Copy link
Copy Markdown
Member

Prepares the project for publishing Thunderbird Mobile Components releases.

Adds release and snapshot publishing workflows, BOM/release/publishing Gradle tasks, release tag and version validation support, and maintainer release documentation.

Also enables ABI validation, Dokka, Gradle isolated projects, and updates Detekt/build logic for the new release-ready setup.

Bumps dependencies to be on the same state as TfA. This includes migration to AGP 9.

@wmontwe wmontwe requested a review from a team as a code owner June 19, 2026 16:41
@wmontwe wmontwe requested a review from jbott-tbird June 19, 2026 16:41
@wmontwe wmontwe added the pr: stacked Must be used on a PR that is stacked on top of other(s) label Jun 19, 2026
@wmontwe wmontwe requested review from rafaeltonholo and removed request for jbott-tbird June 19, 2026 16:42
@wmontwe wmontwe force-pushed the chore-build-prepare-for-releases branch 2 times, most recently from ab0dee1 to 3ce3f0c Compare June 22, 2026 09:33
Comment thread config/detekt/detekt.yml
value: 'STOPSHIP:'
- reason: 'Forbidden TODO todo marker in comment, please do the changes or create an issue to address it.'
value: 'TODO:'
allowedPatterns: '(TODO|FIXME)\((@[a-zA-z0-9-]+|#[0-9-]+)\):'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would keep this. This allow us to add follow-up tasks without missing it

@wmontwe wmontwe Jun 23, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Comment thread config/detekt/detekt.yml
- 'equals'
excludeLabeled: false
excludeReturnFromLambda: true
excludeGuardClauses: true

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would keep this true. Guard clauses are very useful for pre-requisits validation and helps with redability.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Needs a patch on TfA

Comment thread docs/release-guide.md
@wmontwe wmontwe force-pushed the chore-build-prepare-for-releases branch from 3ce3f0c to 0510747 Compare June 23, 2026 15:51
@wmontwe wmontwe force-pushed the chore-build-prepare-for-releases branch from bbdc6d4 to b6396a3 Compare June 24, 2026 11:49
@wmontwe wmontwe removed the pr: stacked Must be used on a PR that is stacked on top of other(s) label Jun 24, 2026
@wmontwe wmontwe requested a review from rafaeltonholo June 24, 2026 12:49

@rafaeltonholo rafaeltonholo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGMT. Just a minor comment, but feel free to merge the changes

Comment thread .github/dependabot.yml
@wmontwe wmontwe merged commit 5b7d3f0 into thunderbird:main Jun 25, 2026
11 checks passed
@wmontwe wmontwe deleted the chore-build-prepare-for-releases branch June 25, 2026 11:05
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