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

Updated README #4920

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

Updated README #4920

wants to merge 26 commits into from

Conversation

0x5bfa
Copy link

@0x5bfa 0x5bfa commented Nov 27, 2024

@niels9001
Copy link

@0x5bfa Love this! Great to bring consistency across the different WinUI/WASDK repo's.

I was looking at this last week myself: https://github.com/niels9001/WindowsAppSDK

Since we now have an official WASDK logo, you might want to copy the assets from my PR?

@0x5bfa
Copy link
Author

0x5bfa commented Nov 27, 2024

Oh didn't notice, for sure. I'll leave to you!
I tried to make it look better in my pr.

@0x5bfa 0x5bfa closed this Nov 27, 2024
@0x5bfa 0x5bfa deleted the 5bfa/GH-ReadMe branch November 27, 2024 15:26
@0x5bfa 0x5bfa restored the 5bfa/GH-ReadMe branch November 28, 2024 09:53
@0x5bfa 0x5bfa reopened this Nov 28, 2024
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
0x5bfa and others added 12 commits November 28, 2024 22:58
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
Co-authored-by: Niels Laute <[email protected]>
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
0x5bfa and others added 2 commits November 29, 2024 00:44
Copy link

@niels9001 niels9001 left a comment

Choose a reason for hiding this comment

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

LGTM!

README.md Outdated Show resolved Hide resolved
0x5bfa and others added 2 commits November 29, 2024 04:11
@0x5bfa
Copy link
Author

0x5bfa commented Dec 26, 2024

So what blocks this pr, I don't see any reason?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants