Skip to content

feat(ipa): Commit checks and IPA changelog #852

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 19 commits into from
Jul 30, 2025
Merged

feat(ipa): Commit checks and IPA changelog #852

merged 19 commits into from
Jul 30, 2025

Conversation

sphterry
Copy link
Contributor

@sphterry sphterry commented Jul 30, 2025 •
edited
Loading

Proposed changes

  • Semantic commit checks on all PRs to the repo
    - updated releases to be semantic commits
    - tested in draft PR
  • IPA commit check to enforce (ipa) scope on PRs to ipa package
    - updated dependabot to use correct scope
    - tested in draft PR
  • Changelog generation for ipa package
    - empty on first release
    - only includes commits with (ipa) scope
  • Enforce changelog update if and only if the package version is bumped
    - tested in draft PR

Jira ticket: CLOUDP-329998

@sphterry sphterry changed the title DRAFT: commit checks and IPA changelog feat: commit checks and IPA changelog Jul 30, 2025
@sphterry sphterry changed the title feat: commit checks and IPA changelog feat: Commit checks and IPA changelog Jul 30, 2025
@sphterry sphterry changed the title feat: Commit checks and IPA changelog feat(ipa): Commit checks and IPA changelog Jul 30, 2025
@sphterry sphterry marked this pull request as ready for review July 30, 2025 11:27
@sphterry sphterry requested review from a team as code owners July 30, 2025 11:27
lovisaberggren
lovisaberggren reviewed Jul 30, 2025
View reviewed changes
lovisaberggren
lovisaberggren approved these changes Jul 30, 2025
View reviewed changes
Copy link
Collaborator

@lovisaberggren lovisaberggren left a comment

Choose a reason for hiding this comment

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

LGTM!

andreaangiolillo
andreaangiolillo reviewed Jul 30, 2025
View reviewed changes
wtrocki
wtrocki reviewed Jul 30, 2025
View reviewed changes
wtrocki
wtrocki reviewed Jul 30, 2025
View reviewed changes
wtrocki
wtrocki reviewed Jul 30, 2025
View reviewed changes
wtrocki
wtrocki reviewed Jul 30, 2025
View reviewed changes
@sphterry sphterry merged commit 93fc904 into main Jul 30, 2025
11 checks passed
@sphterry sphterry deleted the CLOUDP-329998-changelog branch July 30, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andreaangiolillo andreaangiolillo andreaangiolillo left review comments

@wtrocki wtrocki wtrocki approved these changes

@lovisaberggren lovisaberggren lovisaberggren approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sphterry @wtrocki @andreaangiolillo @lovisaberggren