-
-
Notifications
You must be signed in to change notification settings - Fork 76
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
Improve CI #43
base: master
Are you sure you want to change the base?
Improve CI #43
Conversation
@nyalldawson please, don't be cross with me, but I have to put a break on your efforts for the moment. For the new pull request, following this one, I'll have a look later, when I have finished my changes. I understand your intention of keeping things separate in branches. But with this amount of changes, touching the same file in multiple branches, makes merging a pain, lots of conflicts. So we possibly find a better way to handle this with less efforts. I have some ideas, but no time yet, must run. |
No problem, I totally understand! I'll hold off on the prs until you're ready |
@nyalldawson I finally found the time to push my local merging and conflict solving. Before I can come back to any new pull requests I have to handle some urgent vulnerability reports before Oct. 31. Many thanks for your efforts! |
Given the heavy improvements currently being done, it makes sense to improve the CI first 🥳
This PR:
https://github.com/qgis/libdxfrw also has a workflow for periodic builds using coverity scan, but that workflow isn't so easy to port (requires a repo maintainer to add the private token for the build submission, and will first require coverity setup for this repo. I can do that if desired, but I'll still need someone to add the private token to this repo)