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

.vscode/settings.json should not have changes (be "dirty") after running (initial) builds #545

Open
lbovet opened this issue Feb 25, 2024 · 3 comments
Assignees
Labels
⏳awaiting reporter Committers are waiting for reaction from reporter build contributors Affects Contributors (not users)

Comments

@lbovet
Copy link
Contributor

lbovet commented Feb 25, 2024

Expected Behavior

No changed files after initial build

Actual Behavior

.vscode/settings.json has changes, maybe IDE-generated files should not be under source control

  • Version: 2ce7768
  • Platform: Ubuntu 20.04
@vorburger vorburger self-assigned this Feb 25, 2024
@vorburger vorburger added the contributors Affects Contributors (not users) label Feb 25, 2024
@vorburger
Copy link
Member

vorburger commented Feb 25, 2024

maybe IDE-generated files should not be under source control

May I disagree with this? Enola's .vscode/settings.json contains a number of settings which are required to work on this project with VSC. There is no "other source" for such settings (that I know of) which "duplicates" these, or from which they could be "generated". As such, my view would be that this file does belong under source control. Would you still disagree, given this additional new information for context?

No changed files after initial build Actual Behavior .vscode/settings.json has changes,

Ah, but that I do agree with! 😄 Let's perhaps re-focus this issue on seeing if we can do anything to improve that - OK for you?

What is "dirty", what's the "diff" that you are seeing, after an "initial build"? (Are you running ./test.bash for that?)

PS: You technically ARE free to use another IDE than VSC, if you prefer!

@vorburger vorburger added build ⏳awaiting reporter Committers are waiting for reaction from reporter labels Feb 25, 2024
@vorburger vorburger changed the title .vscode/settings.json should maybe not be under source control .vscode/settings.json should not have changes (be "dirty") after running (initial) builds Feb 25, 2024
@lbovet
Copy link
Contributor Author

lbovet commented Feb 25, 2024

Yes, the issue is the dirty state after build, not having the file under source control. Just mentioned removing it as possible solution.

Here is the diff:

index 128cd16..90fb0a6 100644
--- a/.vscode/settings.json
+++ b/.vscode/settings.json
@@ -128,10 +128,16 @@
     "**/CVS": true,
     "**/.DS_Store": true,
     "**/Thumbs.db": true,
+    ".gitignore": true,
+    ".pytest_cache": true,
+    ".vscode/": true,
+    "**/__pycache__/": true,
+    "**/*.pyc": true,
+    "**/obj/": true,
     "**/.classpath": true,
-    "**/.factorypath": true,
     "**/.project": true,
     "**/.settings": true,
+    "**/.factorypath": true,
     "**/*.crswap": true,
     "**/eclipse-bin/": true,
     "**/eclipse-testbin/": true,

@vorburger
Copy link
Member

  • "**/.factorypath": true,
  • "**/.factorypath": true,

This ^^^ I've also noticed keeps "jumping around" and changing position - and I've no idea why! I think it may be different between different versions of... well, I'm not entirely sure what! Is it VSC itself that changes this? Or is one of the (many) code formatters that I have set up in this project perhaps doing this? E.g. slightly different local versions of... prettier - maybe? I would also like to fix that, and welcome investigations into how!

  • ".gitignore": true,
  • ".vscode/": true,

This is very curious, because it's ".vscode": false, here... which VSC version do you use? I have 1.86.2..

  • ".pytest_cache": true,
  • "**/pycache/": true,
  • "**/*.pyc": true,
  • "**/obj/": true,

These I have never seen! You probably have a more fancy Python environment than me? Or some Python related VSC extensions? Would you like to raise a PR to add at least these 4 already? I'm happy to merge that ASAP! (Maybe let's already do just these 4, to start with; I think if you raise a PR for the other ones they will just keep getting changed again by future changes from me.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⏳awaiting reporter Committers are waiting for reaction from reporter build contributors Affects Contributors (not users)
Projects
None yet
Development

No branches or pull requests

2 participants