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

WIP: Update tclap #132

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

WIP: Update tclap #132

wants to merge 2 commits into from

Conversation

hjmjohnson
Copy link
Member

CI fails because the dockerhub image is out of date.

hjmjohnson and others added 2 commits September 5, 2020 09:15
git clone https://git.code.sf.net/p/tclap/code

Manually merge changes from d6419ab71623821c5dd02be3bc2835577fa7d525
@hjmjohnson
Copy link
Member Author

related to #115

@hjmjohnson
Copy link
Member Author

@jcfr The patches for updating the docker image have already been posted #127 , but a new docker image needs to be generated and uploaded to dockerhub with the new version of cmake. Then the WIP patch using the un-official docker image for CI can be removed from this PR.

@hjmjohnson
Copy link
Member Author

@jcfr closing stale pull requests.

@hjmjohnson hjmjohnson closed this Aug 8, 2023
@jcfr jcfr reopened this Aug 8, 2023
@jcfr
Copy link
Member

jcfr commented Aug 8, 2023

Work to finalize this has been initiated. Now that the clang-tidy warnings have been integrated (#135) this will come next.

@hjmjohnson
Copy link
Member Author

@jcfr Thanks. I'm just doing house cleaning. I have too many open issues and pull requests that are slowing down more important work. After 3 years, this is just not all that important to me. :).

Thank you for your continued support and efforts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants