Skip to content

Update gitignore#933

Merged
axxel merged 2 commits intozxing-cpp:masterfrom
Xie-Long-0:master
Apr 22, 2025
Merged

Update gitignore#933
axxel merged 2 commits intozxing-cpp:masterfrom
Xie-Long-0:master

Conversation

@Xie-Long-0
Copy link
Contributor

@Xie-Long-0 Xie-Long-0 commented Apr 15, 2025

Ignore .vs and .vscode files

Copy link
Collaborator

@axxel axxel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems you did not follow the line endings style. Please make sure your lines end the way as all others in this file (unix style line endings).

.gitignore Outdated

# CMake
/build
/out
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where would that directory come from? What is it supposed to contain?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm so sorry, this /build directory is the output directory I specified, it shouldn't appear here, I will delete it.

.gitignore Outdated
# CMake
/build
/out
/CMakePresets.json
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove that file. If present, this is expressly meant to be shared via git.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These files out/ CMakePresets.json CMakeUserPresets.json are automatically generated by opening the project in CMake mode in VS.
The out/ is the compilation output directory, I will change it to somewhere else. But CMakePresets.json bothers me, I can't delete it, and it seems to be automatically generated by the Qt plugin.

// CMakePresets.json
{
  "version": 3,
  "configurePresets": [
    {
      "hidden": true,
      "name": "Qt",
      "cacheVariables": {
        "CMAKE_PREFIX_PATH": "$env{QTDIR}"
      },
      "vendor": {
        "qt-project.org/Qt": {
          "checksum": "wVa86FgEkvdCTVp1/nxvrkaemJc="
        }
      }
    }
  ],
  "vendor": {
    "qt-project.org/Presets": {
      "checksum": "67SmY24ZeVbebyKD0fGfIzb/bGI="
    }
  }
}

@axxel axxel merged commit 98e4219 into zxing-cpp:master Apr 22, 2025
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.

2 participants