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

Portmaster: Resolve CI/CD styling failure #6

Closed
wants to merge 12 commits into from

Conversation

Sntx626
Copy link

@Sntx626 Sntx626 commented Jul 31, 2024

Resolve CI/CD styling failure

@nyabinary
Copy link
Owner

Can't be rebased and merged currently because of conflicts :P

@Sntx626
Copy link
Author

Sntx626 commented Jul 31, 2024

It's fine for me to drop this PR if you could run nixfmt on the portmaster module.

@nyabinary
Copy link
Owner

@Sntx626 do you have a matrix? We should probably have real-time communication, we need to make some changes now because Portmaster new UI uses Tauri instead of Electron.

@nyabinary nyabinary force-pushed the portmaster branch 2 times, most recently from e146773 to 98dfa03 Compare August 3, 2024 14:19
@Sntx626
Copy link
Author

Sntx626 commented Aug 3, 2024

I agree with you, it's cumbersome to chat through PR comments.

You can somewhat reliably catch me on matrix @sntx:matrix.org .

I think we opened up an empty chat a while ago.
I just send that contact a message.

If you didn't receive one, feel free to let me know or just message me on matrix.

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