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

Clean up saluki submodule #553

Merged
merged 3 commits into from
Nov 9, 2023
Merged

Clean up saluki submodule #553

merged 3 commits into from
Nov 9, 2023

Conversation

jlaitine
Copy link

@jlaitine jlaitine commented Nov 8, 2023

No description provided.

@jlaitine jlaitine force-pushed the clean_up_saluki_submodule branch from 9a19f3a to 2fb7832 Compare November 8, 2023 14:01
@jlaitine
Copy link
Author

jlaitine commented Nov 8, 2023

This structure doesn't work out in practice, submodule structure still needs to be flattened.

@jlaitine jlaitine force-pushed the clean_up_saluki_submodule branch 2 times, most recently from 3d88be5 to dafd02d Compare November 9, 2023 07:34
@jlaitine jlaitine force-pushed the clean_up_saluki_submodule branch from dafd02d to 24479d3 Compare November 9, 2023 07:43
@jlaitine jlaitine requested review from t-salminen and pussuw November 9, 2023 07:50
@jlaitine
Copy link
Author

jlaitine commented Nov 9, 2023

Note: coverity scan test failing is not due to this PR, I managed to screw up the coverity scan server with clever submodule updates earlier ;)

@jlaitine
Copy link
Author

jlaitine commented Nov 9, 2023

I am pushing in the submodule changes and this PR; seems to be working ok. Will do the same chages to v1.13.3-fc-only branch

@jlaitine jlaitine merged commit 897d1fe into main Nov 9, 2023
15 checks passed
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