Skip to content

Add write permissions to sync-metadata workflow#33

Merged
clampr merged 2 commits intomainfrom
copilot/fix-workflow-commit-issue
Feb 1, 2026
Merged

Add write permissions to sync-metadata workflow#33
clampr merged 2 commits intomainfrom
copilot/fix-workflow-commit-issue

Conversation

Copy link
Contributor

Copilot AI commented Feb 1, 2026

The sync-metadata workflow could not push commits. It had git add and git commit commands but lacked repository write permissions.

Changes

  • Added permissions: contents: write to .github/workflows/sync-metadata.yml
name: Sync Metadata

on:
  workflow_dispatch:

permissions:
  contents: write

jobs:
  sync:
    # ...

The workflow can now push automated updates to static/parameters.json and static/providers.json.

Original prompt

The workflow is not committing the changes. Probably because it doesn't git add the changes. Pls fix.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: clampr <12759785+clampr@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix workflow not committing changes Add write permissions to sync-metadata workflow Feb 1, 2026
Copilot AI requested a review from clampr February 1, 2026 08:27
@clampr clampr marked this pull request as ready for review February 1, 2026 10:32
@clampr clampr requested review from Copilot and removed request for clampr February 1, 2026 10:32
@clampr clampr merged commit 7c8c9f4 into main Feb 1, 2026
5 checks passed
@clampr clampr deleted the copilot/fix-workflow-commit-issue branch February 1, 2026 10:33
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds write permissions to the sync-metadata workflow to enable it to push automated commits. The workflow generates static/parameters.json and static/providers.json files from the meteostat-python repository and commits them back to the docs repository, but it previously lacked the necessary GitHub Actions permissions to push those commits.

Changes:

  • Added permissions: contents: write to the workflow-level permissions in .github/workflows/sync-metadata.yml

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants