actions-gh-pages icon indicating copy to clipboard operation
actions-gh-pages copied to clipboard

feat: default `github_token` to `github.token`

Open MichaelDeBoey opened this issue 4 years ago β€’ 8 comments

Fixes #309

MichaelDeBoey avatar Jun 18 '21 11:06 MichaelDeBoey

Thank you for the work. This pull-request fixes #309

peaceiris avatar Aug 10 '21 17:08 peaceiris

Codecov Report

Merging #624 (4509111) into main (2c9f853) will not change coverage. The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #624   +/-   ##
=======================================
  Coverage   61.73%   61.73%           
=======================================
  Files           4        4           
  Lines         277      277           
  Branches       49       49           
=======================================
  Hits          171      171           
  Misses        105      105           
  Partials        1        1           
Impacted Files Coverage Ξ”
src/set-tokens.ts 43.24% <0.00%> (ΓΈ)

Continue to review full report at Codecov.

Legend - Click here to learn more Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data Powered by Codecov. Last update 2c9f853...4509111. Read the comment docs.

codecov[bot] avatar Sep 09 '21 04:09 codecov[bot]

@peaceiris I think this one can be merged?

MichaelDeBoey avatar Sep 09 '21 08:09 MichaelDeBoey

I have added #309 to the v4 milestone. Since it takes a long time to publish, I will investigate this first from the viewpoint of backward compatibility, then try to merge this into the v3 line. πŸ‘

peaceiris avatar Sep 09 '21 08:09 peaceiris

@peaceiris this PR is backwards compatible, as the action now defaults to github.token when it's not provided. If people provide it themselves, it will still work.

MichaelDeBoey avatar Sep 09 '21 14:09 MichaelDeBoey

this PR is backwards compatible, as the action now defaults to github.token when it's not provided. If people provide it themselves, it will still work.

Not enough. We need to consider the source code. Perhaps, 631dd15 enables backward compatibility.

peaceiris avatar Sep 09 '21 14:09 peaceiris

I understand this change is not backwards compatible because of the case where users are using personal_token instead of github_token (so this is empty).

AFAIK, using a PAT or GITHUB_TOKEN is interchangeable (just different permissions).

Would it be possible to eventually consolidate these two inputs into a token that defaults to github.token?

privatenumber avatar Dec 04 '21 22:12 privatenumber

Could we promote this? I saw some code and the GitHub token sws non required. Would be glad to help here if I can (as it seems straightforward)

georgettica avatar Apr 22 '22 12:04 georgettica