Skip to content

Pretty table headers#13

Merged
brancengregory merged 3 commits intomainfrom
12-pretty-table-headers
Apr 20, 2025
Merged

Pretty table headers#13
brancengregory merged 3 commits intomainfrom
12-pretty-table-headers

Conversation

@brancengregory
Copy link
Member

No description provided.

@brancengregory brancengregory requested a review from Copilot April 20, 2025 06:31
@brancengregory brancengregory linked an issue Apr 20, 2025 that may be closed by this pull request
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 pull request updates the NEWS.md file to document the new release of ojothemes 1.1.0.

  • Adds release notes with the new version and date
  • Mentions an update to gt_* functions regarding Title Case formatting using {snakecase}
Files not reviewed (3)
  • DESCRIPTION: Language not supported
  • R/ojo_gt.R: Language not supported
  • ojothemes.Rproj: Language not supported
Comments suppressed due to low confidence (1)

NEWS.md:3

  • [nitpick] There's a potential ambiguity between 'Title Case' and the use of {snakecase} in the description. Please clarify whether the update involves converting table headers to Title Case or applying snake case transformation to the gt_* functions.
* Brings clean "Title Case" to gt_* functions using `{snakecase}`

@brancengregory brancengregory requested a review from Copilot April 20, 2025 06:32
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 pull request updates the NEWS.md file to announce version 1.1.0 of ojothemes, highlighting a new feature for pretty table headers.

  • Updated the release notes to mention the implementation of clean "Title Case" for column headers.
  • Noted the use of {snakecase} in the header generation process.
Files not reviewed (3)
  • DESCRIPTION: Language not supported
  • R/ojo_gt.R: Language not supported
  • ojothemes.Rproj: Language not supported
Comments suppressed due to low confidence (1)

NEWS.md:3

  • [nitpick] The release note description is a bit ambiguous since it mentions both 'Title Case' and the usage of {snakecase}. Consider clarifying whether {snakecase} is used for formatting or if a separate mechanism applies to produce Title Case headers.
* Brings clean "Title Case" to column headers created by gt_* functions using `{snakecase}`

@brancengregory brancengregory merged commit a5b8b78 into main Apr 20, 2025
1 check passed
@brancengregory brancengregory deleted the 12-pretty-table-headers branch April 20, 2025 06:46
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.

Pretty table headers

2 participants