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

erfadatextra.h #ifndef #87

Merged
merged 2 commits into from
Jan 18, 2022
Merged

Conversation

radonnachie
Copy link
Contributor

Enables source_flattener.py to scrape until #endif and succeed.

- enables source_flattener.py to scrape until #endif
@mhvk
Copy link
Contributor

mhvk commented Jan 17, 2022

@RocketRoss - that looks good and makes sense - we really should have had that (just goes to show how poor my C is...)

Will just check that the tests pass - but I don't see how they could not!

@radonnachie
Copy link
Contributor Author

Just the smallest oversight, happy to help patch. Only came up because of the flatten_sources.py if erfa_fetch. Thanks!

@mhvk
Copy link
Contributor

mhvk commented Jan 17, 2022

@RocketRoss - just noticed something silly, but might as well fix it - there is a missing new-line at the end of the file. Could you fix that? (Happy to do it myself if that's easier)

@radonnachie
Copy link
Contributor Author

Sorted 👍

@avalentino
Copy link
Member

@mhvk I think that you need to approve the test execution in order to run tests

@mhvk
Copy link
Contributor

mhvk commented Jan 18, 2022

@RocketRoss - thanks! Will merge once tests have run again.

@radonnachie
Copy link
Contributor Author

@mhvk Success :)

@mhvk mhvk merged commit 236e557 into liberfa:master Jan 18, 2022
@radonnachie radonnachie deleted the erfadatextra_ifndef branch January 18, 2022 16:36
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