search

lndk-org / lndk

MIT License
76 stars 19 forks source link

Add vscode to gitignore #105

Closed orbitalturtle closed 2 months ago

orbitalturtle commented 2 months ago

@carlaKC Ah looks like github actions uses the most up-to-date clippy, which introduced a new lint. Fixed the build error by moving the attribute in another commit.

This has come up a few times, so maybe this is worth a quick discussion. We could just keep updating new clippy things as they come up in github actions, -or- we could pin the rust/clippy versions in github actions and update only when we bump the rust version ourselves?

carlaKC commented 2 months ago

We could just keep updating new clippy things as they come up in github actions, -or- we could pin the rust/clippy versions in github actions and update only when we bump the rust version ourselves?

Let's pin the version, seems like the most sustainable way forward.