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

fail some build steps if changed or unversioned files exist at the end / several related cleanups and adjustments #6787

Merged
merged 12 commits into from
Dec 15, 2024

Conversation

firewave
Copy link
Collaborator

@firewave firewave commented Sep 9, 2024

No description provided.

@firewave
Copy link
Collaborator Author

firewave commented Sep 9, 2024

This is to expose tests (or other commands) which generate (and leave) files in the source tree. Files which are being ignored should not fail this.

This should primarily be fixed by using temporary folders. In case this can't be done we need to make sure they are cleaned up - and in some cases we might also need to clean previously existing files beforehand.

@firewave firewave changed the title CI-unixish.yml: fail build step if unversioned files exists at the end CI-unixish.yml: fail build step if unversioned files exist at the end Sep 9, 2024
@firewave

This comment was marked as resolved.

@firewave
Copy link
Collaborator Author

HEAD detached at pull/6787/merge
Untracked files:
  (use "git add <file>..." to include in what will be committed)
	.ccache/
	addons/test/cppcheck-addon-ctu-file-list
	cmake.output.signal/
	cppcheck-addon-ctu-file-list
	test-signalhandler
	test-stacktrace
	test/cli/cppcheck-addon-ctu-file-list
	test/cli/helloworld/cppcheck-addon-ctu-file-list
	test/cli/helloworld/test.cppcheck
	test/cli/proj2/test.cppcheck
	test/cli/test.cppcheck
nothing added to commit but untracked files present (use "git add" to track)

@firewave firewave changed the title CI-unixish.yml: fail build step if unversioned files exist at the end CI-unixish.yml: fail build step if changed and unversioned files exist at the end Oct 30, 2024
@firewave firewave changed the title CI-unixish.yml: fail build step if changed and unversioned files exist at the end CI-unixish.yml: fail build step if changed or unversioned files exist at the end Oct 30, 2024
@firewave firewave changed the title CI-unixish.yml: fail build step if changed or unversioned files exist at the end CI-unixish.yml: fail build step if changed or unversioned files exist at the end / several related cleanups and adjustments Dec 2, 2024
@firewave firewave changed the title CI-unixish.yml: fail build step if changed or unversioned files exist at the end / several related cleanups and adjustments fail some build steps if changed or unversioned files exist at the end / several related cleanups and adjustments Dec 2, 2024
@firewave firewave force-pushed the uknown-files branch 3 times, most recently from 789f711 to 2c5873e Compare December 2, 2024 15:12
@firewave firewave force-pushed the uknown-files branch 2 times, most recently from 035f2ae to d35a9bc Compare December 11, 2024 19:03
@firewave firewave marked this pull request as ready for review December 11, 2024 19:03
@firewave firewave force-pushed the uknown-files branch 2 times, most recently from 2c0a49f to 0cc1a98 Compare December 13, 2024 11:44
@firewave firewave merged commit 1093382 into danmar:main Dec 15, 2024
60 checks passed
@firewave firewave deleted the uknown-files branch December 15, 2024 18:44
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