Skip to content

Bring 'main' up to current state of 8.3 branch#19

Closed
briskt wants to merge 37 commits intomainfrom
8.3
Closed

Bring 'main' up to current state of 8.3 branch#19
briskt wants to merge 37 commits intomainfrom
8.3

Conversation

@briskt
Copy link
Contributor

@briskt briskt commented Apr 8, 2025

No description provided.

Baggerone and others added 30 commits February 15, 2023 08:54
Just adding something to force a difference

Approved-by: Mark Tompsett
Install `jq` for ease in diffing composer updates

Approved-by: S Bagwell
Approved-by: Jason Jackson
Get updated encrypted env file, to fix CI/CD builds for `8.1` branch
Fix corrupted whenavail script (branch 8.1)
set the read bit on s3-expand and whenavail
@briskt briskt requested review from a team, iceman198, jason-jackson and mtompset and removed request for a team April 8, 2025 07:22
@briskt
Copy link
Contributor Author

briskt commented Apr 10, 2025

@jason-jackson do you think this repo even needs a main branch?

@jason-jackson
Copy link
Contributor

@jason-jackson do you think this repo even needs a main branch?

No, I don't. All this PR will do is push a main tag to our docker hub, which isn't helpful. I'd be in favor of just removing main and using PHP version numbers only (and changing default branch as needed).

Copy link

@mtompset mtompset left a comment

Choose a reason for hiding this comment

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

Since Jason followed up on his question, I can approved in good conscience now.

@briskt briskt deleted the branch main April 11, 2025 01:45
@briskt briskt closed this Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants