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

chore: No more painfully to see warnings #357

Merged
merged 7 commits into from
Dec 7, 2023

Conversation

svrooij
Copy link
Contributor

@svrooij svrooij commented Oct 2, 2023

It's just that ☝️ I cleared up the build warnings so your github actions result will no longer look like:

image

@svrooij svrooij requested a review from a team as a code owner October 2, 2023 16:40
@waldekmastykarz
Copy link
Collaborator

Thank you! We'll have a look asap

@svrooij
Copy link
Contributor Author

svrooij commented Dec 4, 2023

@waldekmastykarz I got an update that you merged the latest main to this branch, but there aren’t any updates right?

@waldekmastykarz
Copy link
Collaborator

I got a message that there were updates on main so added them. I'll have a closer look at the PR asap 👍

@waldekmastykarz waldekmastykarz self-assigned this Dec 7, 2023
@waldekmastykarz waldekmastykarz merged commit 8efce67 into microsoft:main Dec 7, 2023
4 checks passed
@waldekmastykarz
Copy link
Collaborator

Thank you so much for the improvement @svrooij 👏

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.

3 participants