Skip to content

feat(spell-check): replace spell-check dictionaries to use AWF repositories#312

Merged
xmfcx merged 2 commits into
autowarefoundation:mainfrom
mitsudome-r:replace-spell-check-dict
Dec 18, 2025
Merged

feat(spell-check): replace spell-check dictionaries to use AWF repositories#312
xmfcx merged 2 commits into
autowarefoundation:mainfrom
mitsudome-r:replace-spell-check-dict

Conversation

@mitsudome-r

Copy link
Copy Markdown
Member

Description

This modifies spell-check actions to use the dictionaries from AWF repository instead of TIER IV dictionaries so that we have full control over addition/deletion of words in the dictionaries without waiting approval from TIER IV.

Tests performed

Not applicable.

Effects on system behavior

Not applicable.

Interface changes

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@mitsudome-r mitsudome-r requested a review from xmfcx August 27, 2024 08:39

@Shin-kyoto Shin-kyoto left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks!! LGTM.
Can you change this workflow to required?

@stale

stale Bot commented Oct 28, 2024

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity.

@stale stale Bot added the status:stale label Oct 28, 2024
Comment thread spell-check/action.yaml Outdated
@stale

stale Bot commented Jan 30, 2025

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity.

@stale stale Bot added the status:stale label Jan 30, 2025
@stale stale Bot removed the status:stale label Dec 18, 2025
@HansRobo

Copy link
Copy Markdown
Member

@mitsudome-r @xmfcx
What's the status of this pull request?
I think we can merge this pull-request after applying @xmfcx 's patch.

@mitsudome-r

Copy link
Copy Markdown
Member Author

@HansRobo Thanks. I have applied the changes.

@mitsudome-r

Copy link
Copy Markdown
Member Author

@xmfcx I have applied your suggestion so could you re-review this PR?

@mitsudome-r mitsudome-r changed the title feat(spell-check): replace spell-check dictionaries to use AWf repositories feat(spell-check): replace spell-check dictionaries to use AWF repositories Dec 18, 2025
@mitsudome-r mitsudome-r enabled auto-merge (squash) December 18, 2025 08:12
@xmfcx xmfcx disabled auto-merge December 18, 2025 08:13
mitsudome-r and others added 2 commits December 18, 2025 17:15
…tories

Signed-off-by: mitsudome-r <ryohsuke.mitsudome@tier4.jp>
Co-authored-by: Mete Fatih Cırıt <mfc@autoware.org>
@xmfcx xmfcx self-requested a review December 18, 2025 08:16
@xmfcx

xmfcx commented Dec 18, 2025

Copy link
Copy Markdown
Contributor

@mitsudome-r could you rebase/update this pr to the latest main to re-trigger the ci?

@mitsudome-r mitsudome-r force-pushed the replace-spell-check-dict branch from 727f8d7 to e599d59 Compare December 18, 2025 08:17
@xmfcx xmfcx merged commit 9282ba6 into autowarefoundation:main Dec 18, 2025
13 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.

5 participants