Skip to content
This repository has been archived by the owner on Oct 29, 2024. It is now read-only.

chore: fix title format #361

Merged
merged 3 commits into from
Sep 24, 2024
Merged

chore: fix title format #361

merged 3 commits into from
Sep 24, 2024

Conversation

pinglin
Copy link
Member

@pinglin pinglin commented Sep 23, 2024

Because

  • titles should take into account articles, conjunctions, prepositions, and special terms

This commit

  • apply general title rules, e.g., OCR, Image to Text, URL

@donch1989 donch1989 changed the title Pinglin/fix title format fix: title format Sep 24, 2024
@donch1989 donch1989 changed the title fix: title format chore: fix title format Sep 24, 2024
@donch1989 donch1989 merged commit 552d12e into main Sep 24, 2024
11 of 12 checks passed
@donch1989 donch1989 deleted the pinglin/fix-title-format branch September 24, 2024 02:48
jvallesm added a commit to instill-ai/pipeline-backend that referenced this pull request Oct 18, 2024
Because

- We relied on writing property titles with Title Case manually (e.g.
instill-ai/component#381,
instill-ai/component#361).

This commit

- Uses Title Case for property titles, too
- Extends lowercase dictionary
- Extends uppercase dictionary
- Respects capitalization of lowercase words if at the end of the title
- Respects capitalization of uppercase words if pluralized
- Adds unchanged words
- Moves dash (-) and underscore (_) replacement to anchor generation
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants