-
Notifications
You must be signed in to change notification settings - Fork 668
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
feat: require minimum Qt6.7 #12003
feat: require minimum Qt6.7 #12003
Conversation
78534a3
to
d9d6f4e
Compare
d9d6f4e
to
c0d328e
Compare
client build failed because of some deprecation warnings: [239/452] Building CXX object src/gui/CMakeFiles/owncloudGui.dir/activitywidget.cpp.o
/drone/src/src/gui/activitywidget.cpp:90:34: warning: 'qAsConst<QMap<QString, OCC::NotificationWidget *>>' is deprecated: Use std::as_const() instead. [-Wdeprecated-declarations]
90 | for (const auto widget : qAsConst(_widgetForNotifId)) { |
might be the unmatched system libraries in |
I am actually considering to move this step over to GitHub..... |
yeeah, maybe we could rely on ctest in github actions |
@saw-jan you want to work on that? |
yep, I can. I will remove the drone pipeline for ctest in this PR. That means we no longer use |
ctest is already being executed in github workflow - client/.github/workflows/main.yml Line 164 in 9bc3844
|
yeah, |
No description provided.