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

remove import linter #33319

Merged
merged 4 commits into from
Aug 17, 2024
Merged

remove import linter #33319

merged 4 commits into from
Aug 17, 2024
Rate limit · GitHub

Access has been restricted

You have triggered a rate limit.

Please wait a few minutes before you try again;
in some cases this may take up to an hour.

Conversation

sshane
Copy link
Contributor

@sshane sshane commented Aug 17, 2024

No description provided.

@sshane sshane changed the title rm import linter remove import linter Aug 17, 2024
@sshane
Copy link
Contributor Author

sshane commented Aug 17, 2024

@maxime-desroches I couldn't figure out how to update the lockfile without making a bunch of changes to existing package markers, is this normal? 5e46f48

@sshane sshane merged commit bf68eeb into master Aug 17, 2024
16 of 17 checks passed
@sshane sshane deleted the remove-importlinter branch August 17, 2024 06:57
@maxime-desroches
Copy link
Contributor

@sshane Yeah, uv lock is still experimental so sometime when the uv version change between two locking they also change how they format/render the uv.lock file

Edison-CBS pushed a commit to Edison-CBS/openpilot that referenced this pull request Sep 15, 2024
* rm import linter, we're done!

* revert

* uv lock

* Revert "uv lock"

This reverts commit 5e46f48.
old-commit-hash: bf68eeb
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.

Rate limit · GitHub

Access has been restricted

You have triggered a rate limit.

Please wait a few minutes before you try again;
in some cases this may take up to an hour.

2 participants