TheAlgorithms / scripts

Scripts used across all The Algorithms repositories
MIT License
77 stars 43 forks source link

Update copyright license script #22

Closed alexpantyukhin closed 1 year ago

alexpantyukhin commented 1 year ago

add Update copyright license

alexpantyukhin commented 1 year ago

@Panquesito7 This is a good idea! But what actually could be configurable there?

Panquesito7 commented 1 year ago

@Panquesito7 This is a good idea! But what actually could be configurable there?

Hmm, that's right. I guess it's a good idea to stick to this. 👍 To use this in other repositories, the whole file will need to be copied (we might want to explain this in a readme file or something).

tjgurwara99 commented 1 year ago

There are a few other changes that I would like you to make but to explain them over PR would be difficult. I've made those changes in the attached patch file. Just download and do apply the patch to your fork git apply workflow.patch where workflow.patch is the file you downloaded.

workflow.patch

Let me know if you didn't understand something inside this patch 😄

Once you apply the patch, you should have 3 changed files, commit them to your fork, push your changes and run the workflow by going to your Actions tab and the update-license with the dispatch trigger. That way you can see it works the way you expect your workflow to work.

Panquesito7 commented 1 year ago

Would you like to add these in the C/C++ repositories, or should we create an issue to have it open to someone else? @alexpantyukhin.

alexpantyukhin commented 1 year ago

@Panquesito7 I think I can do it. What is the best way to do this? Just copy this script into the .github folder?

Panquesito7 commented 1 year ago

I think we need to it similar to this:

      - name: Filename Formatter
        uses: TheAlgorithms/scripts/formatter@main
        with:
          filetypes: .c,.h

However, let's wait a bit. I think there's something wrong with the script, not 100% sure, though.