classtranscribe / FrontEnd

The React + Redux Frontend for ClassTranscribe
https://classtranscribe.illinois.edu
Other
25 stars 27 forks source link

Warning: The `set-output` command is deprecated #804

Closed harsh183 closed 3 weeks ago

harsh183 commented 1 month ago

The CI says:

Run echo "::set-output name=dir::$(yarn cache dir)

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/

robkooper commented 1 month ago

need to merge main back to staging

harsh183 commented 1 month ago

@robkooper @angrave maybe I'm missing something, but I find the branch structure a bit confusing.

Should we add a note in wiki around the branches? Github seems to label staging as the default branch, but we also have a main. Is that the deploy branch?

angrave commented 1 month ago

Fixed the CI cache (copied the CI.yaml back into stagin) https://github.com/classtranscribe/FrontEnd/blob/staging/.github/workflows/ci.yaml

@harsh183 staging is our development branch. All commits should go there. This is automatically deployed (via docker images sentto dockerhub) to ct-dev (about once an hour).

main is "real" production - seen by students and classes. New images built from main branch are also sent to dockerhub. These are picked up classtranscribe.illinois.edu every hour.

angrave commented 1 month ago

This issue should be fixed. Anyone can close it after at least one PR to staging completes without error / warning.

harsh183 commented 3 weeks ago

Looks like lots of builds made it