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 version and build of cppfront to generated files. #791

Merged
merged 7 commits into from
Dec 25, 2023

Conversation

iprtel
Copy link
Contributor

@iprtel iprtel commented Oct 30, 2023

I would find it useful to track which version of cppfront generated a .cpp or .h file.

@hsutter

This comment was marked as resolved.

@hsutter

This comment was marked as resolved.

@hsutter
Copy link
Owner

hsutter commented Dec 19, 2023

Thanks! I added a flag so that it's opt-in, because likely users won't want this to generate extra diffs just because they updated their compiler.

I noticed that there seemed to be a large diff conflict, and unfortunately when I tried to resolve the conflict it seems to have lost the code change, and when I went back into the branch to restore it git wanted me to pull over 60 commits. I wonder if something got broken, and you should regenerate this PR? Sorry for the busywork, I appreciate the suggestion!

@iprtel
Copy link
Contributor Author

iprtel commented Dec 25, 2023

I had to 'manually cherry pick' your commit. Sorry about that.

@hsutter
Copy link
Owner

hsutter commented Dec 25, 2023

Thanks!

@hsutter hsutter merged commit ec06a73 into hsutter:main Dec 25, 2023
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.

2 participants