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

Add support for pop meetings #414

Merged
merged 5 commits into from
Mar 21, 2022
Merged

Conversation

Temikus
Copy link
Contributor

@Temikus Temikus commented Feb 7, 2022

Status

READY

Description

This adds support for https://pop.com/ remote pairing software that we heavily use.

Checklist

Steps to Test or Reproduce

Simple unit test added but let me know if you need more.

@Temikus
Copy link
Contributor Author

Temikus commented Feb 7, 2022

Re-pushed as I made a small mistake in tests that was caught by the linter.

jgoldhammer
jgoldhammer previously approved these changes Mar 20, 2022
Copy link
Collaborator

@jgoldhammer jgoldhammer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jgoldhammer
Copy link
Collaborator

I want to merge your pull request @Temikus - can you please fix the linting errors? Thx.

@jgoldhammer
Copy link
Collaborator

I want to merge your pull request @Temikus - can you please fix the linting errors? Thx.

I have detected that we have a problem with swiftformat that times out with a specific rule. So I will try to fix this in master first and update this branch afterwards.

@jgoldhammer
Copy link
Collaborator

@Temikus -additionally please add entries to the changelog files! Thx.

The swiftformat issue is fixed in #423, but needs to be merged into master.

@Temikus
Copy link
Contributor Author

Temikus commented Mar 21, 2022

Oh man @jgoldhammer apologies, I think we collided a bit. Feel free to dismiss if my changes to changelog are not correct.

@jgoldhammer jgoldhammer merged commit 2efea35 into leits:master Mar 21, 2022
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.

2 participants