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

[Core] Add check for supported C/C++ versions #144

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

FintasticMan
Copy link
Contributor

No description provided.

@VisenDev
Copy link

VisenDev commented Jan 2, 2025

Why restrict versions newer than C99? I don't think there have been any major breaking changes in the standard.

(If anything, it seems like we should warn about using C89 accidentally, because that will definitely break and give some weird error messages)

@FintasticMan
Copy link
Contributor Author

This allows all C versions C99 or newer, there's a ! not operator in front of the whole condition.

@VisenDev
Copy link

VisenDev commented Jan 2, 2025

Ahaha, I didn't see that, my bad

@FintasticMan
Copy link
Contributor Author

No problem, it's easily missed!

@nicbarker
Copy link
Owner

Works like a charm, amazing work!

@nicbarker nicbarker merged commit a431254 into nicbarker:main Jan 8, 2025
3 checks passed
@FintasticMan FintasticMan deleted the add-version-check branch January 8, 2025 21:34
St0wy pushed a commit to St0wy/clay that referenced this pull request Jan 9, 2025
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.

3 participants