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.

Sign up for GitHub

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

Jump to bottom

Remove utils #228

Open
lishaduck opened this issue Aug 24, 2024 · 3 comments
Open

Remove utils #228

lishaduck opened this issue Aug 24, 2024 · 3 comments
Labels
internals refactor triaging Issue is still being evaluated. PRs not yet accepted.

Comments

@lishaduck
Copy link
Contributor

lishaduck commented Aug 24, 2024 •
edited
Loading

Utils is fully unused. It ought to be deleted.
@lishaduck lishaduck changed the title Remove sheriff-utils Remove utils, and bundle types Aug 24, 2024
@lishaduck lishaduck changed the title Remove utils, and bundle types Refactor packages Aug 24, 2024
@lishaduck lishaduck changed the title Refactor packages Refactor package structure Aug 24, 2024
@AndreaPontrandolfo
Copy link
Owner

i'd prefer to keep the things 2 separate issues

@lishaduck lishaduck changed the title Refactor package structure Remove utils Aug 24, 2024
@lishaduck
Copy link
Contributor Author

See also: #233

@AndreaPontrandolfo AndreaPontrandolfo added refactor triaging Issue is still being evaluated. PRs not yet accepted. internals labels Aug 25, 2024
@lishaduck
Copy link
Contributor Author

~Blocked on #229

@lishaduck lishaduck mentioned this issue Nov 1, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
internals refactor triaging Issue is still being evaluated. PRs not yet accepted.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@AndreaPontrandolfo @lishaduck