Skip to content

Conversation

@blaqbern
Copy link
Contributor

@blaqbern blaqbern commented Jul 24, 2025

Copying artifacts a temporary directory so they can be picked up by ncipollo/release-action. This is the last piece that was missing, I think. I ran a dry run successfully and the CHANGELOG, diff and artifacts all looked correct. Also a bit of minor cleanup here as this should be just about finished now.


Related Jira issue: REL-4161: Migrate ld-find-code-refs from Releaser to GHA

@launchdarkly-upra launchdarkly-upra bot changed the title Cblackburn/rel 4161/fix release action config [REL-4161] Cblackburn/rel 4161/fix release action config Jul 24, 2025
@blaqbern blaqbern marked this pull request as ready for review July 24, 2025 22:05
@blaqbern blaqbern requested a review from a team as a code owner July 24, 2025 22:05
@blaqbern blaqbern enabled auto-merge (squash) July 24, 2025 22:05
Copy link

@sheenacarswell sheenacarswell left a comment

Choose a reason for hiding this comment

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

Approving as this seems plausible, but I'm not an expert here

@blaqbern blaqbern merged commit 5b1714b into main Jul 24, 2025
2 checks passed
@blaqbern blaqbern deleted the cblackburn/REL-4161/fix-release-action-config branch July 24, 2025 22:11
Comment on lines -63 to +67
- name: Commit changes and tag
run: |
./scripts/release/commit-and-tag.sh
- name: Create Github release
ls -1a "$ARTIFACT_DIRECTORY"
- name: commit changes and tag
run: ./scripts/release/commit-and-tag.sh
- name: create Github release
Copy link
Contributor Author

@blaqbern blaqbern Jul 24, 2025

Choose a reason for hiding this comment

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

The diff looks so weird here, but this is just lower-casing the names for consistency and some other minor cleanup/logging. The order is not changing here, even though it kinda looks like it from the diff.

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