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.

Sign up for GitHub

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

Jump to bottom

Clean-up dependencies #333

Merged
merged 76 commits into from
Oct 9, 2023
Merged

Clean-up dependencies #333

merged 76 commits into from
Oct 9, 2023

Conversation

20DM
Copy link
Collaborator

@20DM 20DM commented Aug 22, 2023 •
edited
Loading

This change set is to

  • make sure the package can be built with Conan v2
  • make sure the package can be built with vanilla CMake (+ dependencies from homebrew on MacOS)
  • get rid off GreatCMakeCookOff
  • upgrades a few dependencies to more modern versions, incl. eigen/3.4.0, catch2/3.4.0, spdlog/1.12.0
  • add code tweaks to support interface changes in Catch2
  • add code tweaks to get rid off C++ compile time warnings
  • keep the README in sync.

Closes #331
Closes #298
Closes #290
Closes #288

Goes hand-in-hand with astro-informatics/sopt#418

@20DM 20DM added enhancement WIP Still work in progress labels Aug 22, 2023
@20DM 20DM requested a review from mmcleod89 August 22, 2023 16:16
@20DM 20DM self-assigned this Aug 22, 2023
20DM and others added 22 commits August 24, 2023 09:07
@20DM
pointless lint changes
bc9da27
@20DM
tweaks for linux
101d6ab
@20DM
MacOS build tweaks and pfitsio reader clean-up
a4ddbe2
@20DM
apply nonsense linting policy
15bbd6a
@20DM
enough of this
1bda4d9
@20DM
enough of this
ad1854c
@20DM
fix headlines
b3282ae
@20DM
fix headlines
37f1d8a
@20DM
fix headlines
4e1ec50
@20DM
fix headlines
b508841
@20DM
fix headlines
6663976
@20DM
add more eigen spelling variations|
74956b3
@20DM
Merge remote-tracking branch 'origin/development' into chrisg_build_t…
0f4c10e 
…weaks
@20DM
Merge branch 'chrisg_build_tweaks' of github.com:astro-informatics/pu…
3ab29c4 
…rify into chrisg_build_tweaks
@20DM
check either spelling before throwing
d84cb14
@20DM
fix typo in README
dbc809c
@20DM
generate deps
1dcd2d3
@20DM
harmonise sopt spelling
b782ba5
Fix segfault and reinstate algo_factory test
dbf3877
@20DM
re-enable MPI test
232d256
@20DM
add namespace alias for casacore
c1c1131
@20DM
make Conan optional in CI and add alternatice CMake-based route
65a41ce
@20DM 20DM marked this pull request as ready for review October 2, 2023 10:38
@20DM 20DM added continuous-integration and removed WIP Still work in progress labels Oct 3, 2023
mmcleod89
mmcleod89 reviewed Oct 9, 2023
View reviewed changes
README.md Outdated Show resolved Hide resolved
mmcleod89
mmcleod89 reviewed Oct 9, 2023
View reviewed changes
README.md Outdated Show resolved Hide resolved
20DM and others added 2 commits October 9, 2023 13:53
@20DM @mmcleod89
Update README.md
ccab6c2 
Co-authored-by: mmcleod89 <[email protected]>
@20DM @mmcleod89
Update README.md
3ce6f55 
Co-authored-by: mmcleod89 <[email protected]>
@mmcleod89 mmcleod89 merged commit 63a1955 into development Oct 9, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mmcleod89 mmcleod89 mmcleod89 approved these changes

Assignees

@20DM 20DM

Labels
continuous-integration enhancement
Projects
None yet
Milestone
No milestone
2 participants
@20DM @mmcleod89