pret / pokecrystal

Disassembly of Pokémon Crystal
https://pret.github.io/pokecrystal/
2.07k stars 778 forks source link

Github Workflow `set-output` command is deprecated & disabled soon! #1038

Closed vulcandth closed 1 year ago

vulcandth commented 1 year ago

The Github actions workflows are reporting the following error:

Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Supposedly set-output will be disabled May 31st 2023 according the the article https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

I'm not as familiar with the workflows, and I don't exactly see where we are using the command... but someone should take a look.

You can see and example of the above warning by looking at the workflow Update Symbols.

Rangi/Pfero can one of ya'll take a look if you have time?

Rangi42 commented 1 year ago

From https://github.com/EndBug/add-and-commit/issues/448:

You're using an outdated version of add-and-commit Please upgrade to v9, where this issue has already been fixed.

vulcandth commented 1 year ago

Ah that makes sense.

Looks like we will need to do a bit more than just update the version, branch is no longer a valid input. I think we need to switch to the symbols branch first before running this now?

Warning: Unexpected input(s) 'branch', valid inputs are ['add', 'author_name', 'author_email', 'commit', 'committer_name', 'committer_email', 'cwd', 'default_author', 'fetch', 'message', 'new_branch', 'pathspec_error_handling', 'pull', 'push', 'remove', 'tag', 'tag_push', 'github_token']
vulcandth commented 1 year ago

Alright just confirmed on one of my other repo's the branch input is not necessary as we already have the correct branch checked out in the cwd : ./symbols directory.

      - name: Update symbols
        if: ${{ github.event_name == 'push' && github.repository_owner == 'pret' }}
-        uses: EndBug/add-and-commit@v7
+        uses: EndBug/add-and-commit@v9
        with:
-          branch: symbols
          cwd: "./symbols"
          add: "*.sym"
          message: ${{ github.event.commits[0].message }}