Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Add token, repo and sha input" #24

Merged
merged 1 commit into from
Oct 27, 2023
Merged

Conversation

WebFreak001
Copy link
Owner

Reverts #23

use environment variables to customize the behavior instead:

      - name: Deploy Windows release
        if: matrix.os == 'windows-latest'
        uses: WebFreak001/[email protected]
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # automatically provided by github actions
          GITHUB_SHA: 547fdf5 # optional, a commit SHA
          GITHUB_REPOSITORY: Pure-D/serve-d # optional, the repository where the release is located
        with:
          upload_url: https://uploads.github.com/repos/Pure-D/serve-d/releases/20717582/assets{?name,label} # find out this value by opening https://api.github.com/repos/<owner>/<repo>/releases in your browser and copy the full "upload_url" value including the {?name,label} part
          release_id: 20717582 # same as above (id can just be taken out the upload_url, it's used to find old releases)
          asset_path: ./myapp.zip # path to archive to upload
          asset_name: myapp_windows-nightly-$$.zip # name to upload the release as, use $$ to insert date (YYYYMMDD) and 6 letter commit hash
          asset_content_type: application/zip # required by GitHub API
          max_releases: 7 # optional, if there are more releases than this matching the asset_name, the oldest ones are going to be deleted

@WebFreak001 WebFreak001 merged commit 8edd321 into master Oct 27, 2023
1 check passed
@WebFreak001 WebFreak001 deleted the revert-23-master branch October 27, 2023 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant