Skip to content

Conversation

@Panquesito7
Copy link
Member

Things added/changed:

  • Add a Shell filename formatter script. The script lets you:
    • Choose the desired base directory.
    • Choose between two file types (e.g.: *.cpp, *.hpp).
    • Ignore files and directories to not be checked (filenames do not need to include directories as all directories are checked if they have the same filename).
    • Why add it here? It's already being used in various repositories such as the C/C++, Julia, TypeScript, and hopefully more repositories.

I'm not a Shell expert, so if you have any improvements that can be done, let me know. Thanks. 🙂
CC: @uncomfyhalomacro, you might want to check this as you're better with Shell scripts.

@Panquesito7 Panquesito7 added the enhancement New feature or request label Nov 17, 2022
@Panquesito7 Panquesito7 linked an issue Nov 17, 2022 that may be closed by this pull request
Copy link
Member

@tjgurwara99 tjgurwara99 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Have you considered making these scripts as a GitHub action? That way they would be simply a workflow file with some configuration to tell it which ones to execute...

@Panquesito7
Copy link
Member Author

Looks good to me!

Have you considered making these scripts as a GitHub action? That way they would be simply a workflow file with some configuration to tell it which ones to execute...

That sounds like a good idea! I'll be making one for the Filename Formatter later on. For now, let's keep this as-is.

@Panquesito7
Copy link
Member Author

Will be merging for now. This will make various workflows much shorter.

@Panquesito7 Panquesito7 merged commit 167d60c into main Dec 16, 2022
@Panquesito7 Panquesito7 deleted the filename_formatter branch December 16, 2022 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Add a filename formatter script

3 participants