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

Update to QXmpp 1.8.1 #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

melvo
Copy link
Collaborator

@melvo melvo commented Jul 22, 2024

No description provided.

@flathubbot
Copy link

Started test build 135803

@jbruechert
Copy link
Collaborator

The lints have become a lot stricter, and since not much has happened in this repo lately, so they no longer pass.

Looks like the main issue is that the metadata doesn't specify a default screenshot. Adding type="default" to the screenshot block should help.

    <screenshot type="default">
    </screenshot>

This needs to be done upstream and then backported here as a patch.

@flathubbot
Copy link

Build 135803 failed

@lnjX
Copy link
Collaborator

lnjX commented Jul 24, 2024

I fixed the screenshot thing upstream and released Kaidan 0.9.2 (there were also other bug fixes ready to be released) and created #9.

@melvo melvo changed the title Update to QXmpp 1.7.1 Update to QXmpp 1.8.1 Oct 19, 2024
@flathubbot
Copy link

Started test build 155647

@melvo melvo mentioned this pull request Oct 19, 2024
@flathubbot
Copy link

Build 155647 failed

@jbruechert
Copy link
Collaborator

This needs a backport / fixing of the metainfo first to make the CI pass and by extension allow for merging

@flathubbot
Copy link

Started test build 155807

@flathubbot
Copy link

Build 155807 failed

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.

4 participants