Skip to content

Updated .github/workflows/docker-publish.yml#4

Merged
jdunnink merged 10 commits intomainfrom
feature-cr3miaHS
May 29, 2025
Merged

Updated .github/workflows/docker-publish.yml#4
jdunnink merged 10 commits intomainfrom
feature-cr3miaHS

Conversation

@jdunnink
Copy link
Contributor

Description

Brief description of the changes in this PR.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Performance improvement
  • Refactoring (no functional changes)
  • Test improvements

Related Issues

Fixes #(issue number)

Testing

  • Unit tests pass (go test ./...)
  • Integration tests pass
  • Manual testing completed
  • New tests added for new functionality

Checklist

  • My code follows the project's coding standards
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Screenshots (if applicable)

Add screenshots here if the changes affect the UI or output.

Additional Notes

Any additional information, deployment notes, or concerns.

jdunnink added 10 commits May 29, 2025 12:07
- Change WORKDIR from /build to /app in Dockerfile
- Update COPY command to use /app path consistently
- This should resolve the 'stat /build/cmd/gh-actions-exporter: directory not found' error in GitHub Actions
@jdunnink jdunnink merged commit 64b6af5 into main May 29, 2025
1 check passed
@jdunnink jdunnink deleted the feature-cr3miaHS branch May 31, 2025 11:37
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