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

NullClipboard: Fix warnings #11379

Merged
merged 4 commits into from
Aug 26, 2023

Conversation

kiminuo
Copy link
Collaborator

@kiminuo kiminuo commented Aug 25, 2023

Just fix a few warnings

Copy link
Collaborator

@adamPetho adamPetho left a comment

Choose a reason for hiding this comment

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

Apart from one comment below, LGTM

@@ -1,17 +1,15 @@
using System.Threading.Tasks;
using Avalonia.Input;
using Avalonia.Input.Platform;

namespace WalletWasabi.Fluent.Models.UI;
Copy link
Collaborator

Choose a reason for hiding this comment

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

For me, there is a warning for the namespace also.

I can't add a suggestion here, so code snippet:

using WalletWasabi.Fluent.Models.UI;

namespace WalletWasabi.Tests.UnitTests.ViewModels.UiContext;

Copy link
Collaborator Author

@kiminuo kiminuo Aug 26, 2023

Choose a reason for hiding this comment

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

Yes, I reverted that in this PR actually 479ecfc because changing the namespace leads to a collision with the class name UiContext. So the folder should be renamed I guess. But that renaming can lead to issues with @ichthus1604's refactorings.

@kiminuo kiminuo merged commit bacccfc into WalletWasabi:master Aug 26, 2023
7 checks passed
@kiminuo kiminuo deleted the feature/2023-08-25-NullClipboard branch August 26, 2023 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants