Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add typos to CI checks #440

Closed
wants to merge 16 commits into from
Closed

Add typos to CI checks #440

wants to merge 16 commits into from

Conversation

EricLBuehler
Copy link
Contributor

I added the typos utility to CI, and simplified the enum variants in the BuilderError from #436 .

Description

I added the typos utility to CI (and an exclude file).

I also removed the GEPError as it did not make sense and the 2 others GEPPointee and GEPIndex fit better.

Related Issue

This is related to #438.

It also resolves what @marcospb19 raised in #436.

How This Has Been Tested

CI, including my new addition to it, is passing.

Checklist

@EricLBuehler
Copy link
Contributor Author

EricLBuehler commented Sep 4, 2023

Check failed for spurious reason (network err).

@EricLBuehler
Copy link
Contributor Author

Closing to retry checks.

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.

1 participant