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

XBEL description #811

Merged
merged 2 commits into from
Feb 10, 2025
Merged

XBEL description #811

merged 2 commits into from
Feb 10, 2025

Conversation

sjehuda
Copy link
Contributor

@sjehuda sjehuda commented Jan 9, 2025

As seen in XBEL file of Otter Browser.

I have only tested with the following command.

python buku.py -t instance:xmpp -e list_of_xmpp_servers.xbel

As seen in XBEL file of Otter Browser.
@LeXofLeviafan
Copy link
Collaborator

You may want to update the existing test for this functionality 😅

@sjehuda
Copy link
Contributor Author

sjehuda commented Jan 9, 2025

What do you mean?

@LeXofLeviafan
Copy link
Collaborator

What do you mean?

pull-request check failed
testing log

@sjehuda
Copy link
Contributor Author

sjehuda commented Jan 15, 2025

@LeXofLeviafan

May you please correct or send a new PR?

I do not care for appearing in the commit log.

@LeXofLeviafan
Copy link
Collaborator

…Well, that was a bit tricky (turns out I had to check out the branch named refs/pull/811/head so that I could actually push my changes) 😅

@jarun jarun merged commit bc4307a into jarun:master Feb 10, 2025
1 check passed
@jarun
Copy link
Owner

jarun commented Feb 10, 2025

Thank you!

@sjehuda
Copy link
Contributor Author

sjehuda commented Feb 11, 2025 via email

@github-actions github-actions bot locked and limited conversation to collaborators Mar 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants