Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

switch check out ref to merged commit #998

Merged
merged 1 commit into from
Feb 20, 2025
Merged

Conversation

labkey-willm
Copy link
Contributor

@labkey-willm labkey-willm commented Feb 20, 2025

Rationale

the Copy CSP action was failing due to looking at the ref of the auto-deleted PR branch. This flips that to look at the merged commit sha.

and again make a small space change to application.properties to trigger the action once this PR is merged, to pickup changes from #996

Related Pull Requests

Changes

@labkey-willm labkey-willm merged commit 32152ad into develop Feb 20, 2025
3 of 5 checks passed
@labkey-willm labkey-willm deleted the fb_fix_csp_action_more branch February 20, 2025 17:34
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.

2 participants