Closed gcsecsey closed 1 year ago
I agree with all you've said. I think I was just moving fast while going solo but it doesn't fit for collaboration!
For now what's the recommendation exactly? Delete the vscode config and add to gitignore?
I'm not sure whether we should delete the whole config. For the editor settings, we can go ahead and remove the setting, but for others, it would be better to substitute them with the appropriate git hook or GH action.
I can go over each setting and raise a PR with the easy ones removed, and some others replaced, lmk if that's OK. 🙂
Thanks 🙏🏽 @gcsecsey that makes sense!
Describe the bug Looking at
.vscode/settings.json
, there are numerous workspace settings that change the basic look & feel of the editor. As these settings have no effect on the code pushed to the repo, IMO these should be set on the user level instead of the workspace level.To Reproduce
Looking at some settings in
.vscode/settings.json
:Expected behavior
We should strive to enforce linting/formatting rules on the server side, and/or in pre-commit hooks. This would make contributing to the project easier with other IDEs and independent of IDE settings.