Skip to content

Conversation

@ReaNAiveD
Copy link
Member

  • Support Multi-line Upcoming Breaking Change announcement
  • Bugfix in multi-version report

@ReaNAiveD ReaNAiveD requested review from AllyW and Copilot March 24, 2025 04:17
@ReaNAiveD ReaNAiveD self-assigned this Mar 24, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for multi-line upcoming breaking change announcements and includes a bugfix for the multi-version report.

  • Updated the package version in azdev/init.py
  • Added a new "prepended_detail" property and updated its usage in azdev/operations/breaking_change/init.py to format multi-line details

Reviewed Changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
azdev/init.py Bumped version from 0.1.99 to 0.1.100
azdev/operations/breaking_change/init.py Introduced a new property for formatting multi-line breaking change details and updated its usage in grouping items
Files not reviewed (2)
  • HISTORY.rst: Language not supported
  • azdev/operations/breaking_change/markdown_template.jinja2: Language not supported
Comments suppressed due to low confidence (1)

azdev/operations/breaking_change/init.py:41

  • [nitpick] The current list comprehension includes empty strings for any blank lines in 'self.detail', which may lead to unnecessary blank entries in the output. Consider filtering out empty lines to produce a cleaner concatenated result.
prepended_lines = [f'- {l}' if l else '' for l in self.detail.split('\n')]

@ReaNAiveD ReaNAiveD merged commit 999cfff into Azure:dev Mar 24, 2025
25 checks passed
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