Skip to content

Commit

Permalink
Update sync_docs.yaml variables names discourse_api_username => disco…
Browse files Browse the repository at this point in the history
…urse-api-user(#501)

The discourse_api_username and discourse_api_key are the key names expected by the discourse-gatekeeper action.

The data-platform-workflows version of sync_docs obtains the action's parameters via workflow inputs.
These input keys just don't match the action's keys because I wanted to keep the naming style consistent with other dpw workflow calls. Let's use the DPW variables discourse-api-user and discourse-api-key instead of the action's.
  • Loading branch information
a-velasco authored Jun 18, 2024
1 parent aaa712b commit c8fe970
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/sync_docs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ jobs:
name: Sync docs from Discourse
uses: canonical/data-platform-workflows/.github/workflows/[email protected]
secrets:
discourse_api_username: ${{ secrets.DISCOURSE_API_USERNAME }}
discourse_api_key: ${{ secrets.DISCOURSE_API_KEY }}
discourse-api-user: ${{ secrets.DISCOURSE_API_USERNAME }}
discourse-api-key: ${{ secrets.DISCOURSE_API_KEY }}
permissions:
contents: write # Needed to push branch & tag
pull-requests: write # Needed to create PR

0 comments on commit c8fe970

Please sign in to comment.