Skip to content

Fix/logging changelog update#182

Merged
GaProgMan merged 7 commits into
GaProgMan:mainfrom
jamie-taylor-rjj:fix/logging-changelog-update
Aug 4, 2025
Merged

Fix/logging changelog update#182
GaProgMan merged 7 commits into
GaProgMan:mainfrom
jamie-taylor-rjj:fix/logging-changelog-update

Conversation

@jamie-taylor-rjj

Copy link
Copy Markdown
Collaborator

Rationale for this PR

This PR attempts to fix the current workflow issue related to the following workflow run: https://github.com/GaProgMan/OwaspHeaders.Core/actions/runs/16708912969/job/47290940231

PR Checklist

Feel free to either check the following items (by place an x inside of the square brackets) or by replacing the square brackets with a relevant emoji from the following list:

  • ✅ to indicate that you have checked something off
  • ❎ to indicate that you haven't checked something off
  • ❓ to indicate that something might not be relevant (writing tests for documentation changes, for instance)

Essential

These items are essential and must be completed for each commit. If they are not completed, the PR may not be accepted.

  • [ ❓ ] I have added tests to the OwaspHeaders.Core.Tests project
  • [ ❓ ] I have run the dotnet-format command and fixed any .editorconfig issues
  • [ ✅ ] I have ensured that the code coverage has not dropped below 65%
  • [ ❓ ] I have increased the version number in OwaspHeaders.Core.csproj (only relevant for code changes)
  • ❓[ ] I have updated the changelog in the root of the repository

Note

The changelog in the docs/ directory will be updated automatically when PRs are merged into main.

Optional

  • [ ❓ ] I have documented the new feature in the docs directory
  • [ ❓ ] I have provided a code sample, showing how someone could use the new code

Any Other Information

This section is optional, but it might be useful to list any other information you think is relevant.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 fixes a workflow issue by modifying the changelog synchronization process to create pull requests instead of directly committing changes to the main branch. This approach provides better visibility and control over automated changelog updates.

  • Replaces direct commit and push with automated pull request creation
  • Adds pull-requests write permission to enable PR creation
  • Uses the peter-evans/create-pull-request action for automated PR workflow

Comment thread .github/workflows/sync-changelog.yml
@GaProgMan GaProgMan merged commit f669a19 into GaProgMan:main Aug 4, 2025
4 checks passed
@jamie-taylor-rjj jamie-taylor-rjj deleted the fix/logging-changelog-update branch August 4, 2025 08:10
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