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

feat: append the suffix to the end when generating unique filenames for directories, i.e., after not before the extension #1784

Merged
merged 2 commits into from
Oct 15, 2024

Conversation

Saru2003
Copy link
Contributor

This pull request modifies the folder renaming functionality in the unique_name function. The change ensures that when renaming a folder with a duplicate name, the index is appended at the end of the folder name (e.g., changing a.b.c to a.b.c_1), rather than before the last dot.

This is addressed towards issue #1774, enhancing the usability of the renaming feature by preventing confusion with file names.

Closes: #1774

Please review my implementation and provide feedback.

@sxyazi sxyazi force-pushed the rename-folder-index branch from 96def54 to 92ff55f Compare October 15, 2024 16:39
@sxyazi sxyazi changed the title Change naming convention for folders to append index after the name feat: append the suffix to the end when generating unique filenames for directories, i.e., after not before the extension Oct 15, 2024
Copy link
Owner

@sxyazi sxyazi left a comment

Choose a reason for hiding this comment

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

Thanks!

@sxyazi sxyazi merged commit 17a1169 into sxyazi:main Oct 15, 2024
6 checks passed
@Saru2003
Copy link
Contributor Author

@sxyazi Thank you for merging my pull request. I really appreciate it. If possible, could you please add the 'hacktoberfest-accepted' label to this PR? Thanks again.

@sxyazi
Copy link
Owner

sxyazi commented Oct 16, 2024

Done! @Saru2003

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add index to the end when copy folder
2 participants