Skip to content

Add typos to CI checks#440

Closed
EricLBuehler wants to merge 16 commits intoTheDan64:masterfrom
EricLBuehler:master
Closed

Add typos to CI checks#440
EricLBuehler wants to merge 16 commits intoTheDan64:masterfrom
EricLBuehler:master

Conversation

@EricLBuehler
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

EricLBuehler commented Sep 4, 2023

Check failed for spurious reason (network err).

@EricLBuehler
Copy link
Copy Markdown
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