Skip to content

touch: change message when directory#6506

Merged
sylvestre merged 1 commit intouutils:mainfrom
Its-Just-Nans:touch-directory
Jun 30, 2024
Merged

touch: change message when directory#6506
sylvestre merged 1 commit intouutils:mainfrom
Its-Just-Nans:touch-directory

Conversation

@Its-Just-Nans
Copy link
Contributor

Should fix #2639

@sylvestre
Copy link
Contributor

Please add a test to make sure we don't regress

@Its-Just-Nans
Copy link
Contributor Author

Yes you're right

Now the code is also cleaner I think

@sylvestre
Copy link
Contributor

the initial commit message isn't ideal btw
" change message when directory " :)

@Its-Just-Nans
Copy link
Contributor Author

Can you just squash all commits ? (when merging)

Or I can squash them if needed :)

@sylvestre
Copy link
Contributor

sure but it is still appreciated to have valid commit message. esp as github uses the first as input :)

@Its-Just-Nans
Copy link
Contributor Author

my bad

Is-it correct now ?

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/inotify-dir-recreate (fails in this run but passes in the 'main' branch)

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.

touch no-file/ fails with a sub optimal error

2 participants