Skip to content

chore: resolve merge conflicts - #72

Merged
sbillinge merged 3 commits into
diffpy:skpkg-migrationfrom
SYMSCAE:skpkg-migration
Nov 2, 2026
Merged

sbillinge merged 3 commits into
diffpy:skpkg-migrationfrom
SYMSCAE:skpkg-migration

Conversation

@SYMSCAE

@SYMSCAE SYMSCAE commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

This PR resolves merge conflicts into main brought by #55.

SYMSCAE and others added 3 commits October 18, 2026 18:02
chore: remove customPQConfig and deprecate NoMetaStructureAdapter
# Conflicts:
#	src/diffpy/srreal/EmptyStructureAdapter.cpp
#	src/diffpy/srreal/NoMetaStructureAdapter.cpp
#	src/diffpy/srreal/NoMetaStructureAdapter.hpp
#	src/diffpy/srreal/NoSymmetryStructureAdapter.cpp
#	src/diffpy/srreal/NoSymmetryStructureAdapter.hpp
#	src/diffpy/srreal/PQEvaluator.cpp
#	src/diffpy/srreal/PairQuantity.cpp
#	src/diffpy/srreal/StructureAdapter.hpp
#	src/tests/TestNoMetaStructureAdapter.hpp
#	src/tests/test_custompqconfig.hpp
@SYMSCAE

SYMSCAE commented Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

@sbillinge PR ready for review.

@sbillinge
sbillinge merged commit 2005b2f into diffpy:skpkg-migration Nov 2, 2026
2 checks passed
@sbillinge

Copy link
Copy Markdown
Contributor

yeh, merged! I wonder if we did over the same changes because an earlier issue wasn't closed properly! Anyway, all merged now. @SYMSCAE @cadenmyers13 what is next on the journey to get this out?

@SYMSCAE
SYMSCAE deleted the skpkg-migration branch November 2, 2026 18:50
@SYMSCAE
SYMSCAE restored the skpkg-migration branch November 2, 2026 18:50
@cadenmyers13

Copy link
Copy Markdown
Contributor

@sbillinge I think just merge #71 and we can go through the release checklist

@SYMSCAE

SYMSCAE commented Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

@cadenmyers13 Maybe you can implement the two doc fix quick so that we're cleared codebase-wise?

@SYMSCAE
SYMSCAE deleted the skpkg-migration branch November 3, 2026 16:01
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