Skip to content

Remove cspell.json#6206

Merged
piaskowyk merged 1 commit intomainfrom
@piaskowyk/remove-cspeljson
Jul 8, 2024
Merged

Remove cspell.json#6206
piaskowyk merged 1 commit intomainfrom
@piaskowyk/remove-cspeljson

Conversation

@piaskowyk
Copy link
Member

@piaskowyk piaskowyk commented Jul 3, 2024

Summary

We no longer need this file since we have removed the c-spell CI.

@piaskowyk piaskowyk requested review from szydlovsky and tjzel July 3, 2024 11:32
Copy link
Collaborator

@tjzel tjzel left a comment

Choose a reason for hiding this comment

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

We use this for precommit now 🙈

@piaskowyk
Copy link
Member Author

piaskowyk commented Jul 4, 2024

But where? I can't see any usage of cspell in our codebase 🤔

Copy link
Member

@tomekzaw tomekzaw left a comment

Choose a reason for hiding this comment

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

Goodbye cspell.json

Screenshot 2024-07-04 at 13 24 58

@tjzel
Copy link
Collaborator

tjzel commented Jul 8, 2024

Seems like it got lost between

and

Should I restore it in your opinion or just let it go altogether?

@piaskowyk
Copy link
Member Author

@piaskowyk piaskowyk enabled auto-merge July 8, 2024 07:48
@piaskowyk piaskowyk added this pull request to the merge queue Jul 8, 2024
Merged via the queue into main with commit 99a4ea0 Jul 8, 2024
@piaskowyk piaskowyk deleted the @piaskowyk/remove-cspeljson branch July 8, 2024 08:06
r0h0gg6 pushed a commit to r0h0gg6/react-native-reanimated that referenced this pull request Jul 28, 2025
## Summary

We no longer need this file since we have removed the c-spell CI.
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.

4 participants