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

Disable the unused and problematic tools #124

Merged
merged 1 commit into from
May 28, 2024
Merged

Conversation

mattleibow
Copy link

Description of Change

The Go modules are all far out of date and some tooling is reporting the repo as having security issues. We do not use any part of Go, EMSDK or perfetto in the SkiaSharp project.

In an ideal world we don't have to make changes and there is a special flag that exists for us to "disable all issues", but that does not exist and trying to make that flag work only on CI is unnecessary. We can just delete the few places that are having issues.

The Go modules are all far out of date and some tooling is reporting the
repo as having security issues. We do not use any part of Go, EMSDK or
perfetto in the SkiaSharp project.

In an ideal world we don't have to make changes and there is a special
flag that exists for us to "disable all issues", but that does not exist
and trying to make that flag work only on CI is unnecessary. We can just
delete the few places that are having issues.
@mattleibow mattleibow merged commit bc5cc58 into skiasharp May 28, 2024
1 check passed
@mattleibow mattleibow deleted the dev/disable-problems branch May 28, 2024 19:34
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.

1 participant