Skip to content

Add pr test action#8

Merged
3rob3 merged 2 commits into
immichFrame:mainfrom
bruvv:add-pr-test
Aug 1, 2025
Merged

Add pr test action#8
3rob3 merged 2 commits into
immichFrame:mainfrom
bruvv:add-pr-test

Conversation

@bruvv

@bruvv bruvv commented Jul 26, 2025

Copy link
Copy Markdown
Contributor

How to enable this rule:

  1. Navigate to Settings → Branches.
  2. Under Branch protection rules, click Add rule.
  3. Choose the branch you want to protect (e.g., main).
  4. Enable Require status checks to pass before merging.
  5. Select “Build Check” in the list of required checks.
  6. Save the rule.

@3rob3

3rob3 commented Aug 1, 2025

Copy link
Copy Markdown
Contributor

I apparently missed this PR, sorry and thanks!

@3rob3 3rob3 merged commit 448aeae into immichFrame:main Aug 1, 2025
1 check passed
@3rob3

3rob3 commented Aug 1, 2025

Copy link
Copy Markdown
Contributor

for 3, main isn't an option. "Include default branch" maybe?
for 5, I don't see Build Checks as an option

@bruvv

bruvv commented Aug 2, 2025

Copy link
Copy Markdown
Contributor Author

No worries, here is a video for you to see on how to enable it :) https://calmcode.io/course/github-actions/prevent-merge

@3rob3

3rob3 commented Aug 6, 2025

Copy link
Copy Markdown
Contributor

Still no go. Build, build, or Build Checks does not show as an option.

@bruvv

bruvv commented Aug 7, 2025

Copy link
Copy Markdown
Contributor Author

perhaps this will help:
image

image image image image

@3rob3

3rob3 commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

Yes, did that. As I mentioned build does not show up:
image

@bruvv

bruvv commented Aug 8, 2025

Copy link
Copy Markdown
Contributor Author

that is really weird i do not know sorry @3rob3 :(

@3rob3

3rob3 commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

I got it. You have to run it manually once before it will show (which you must have on your branch). I added a workflow dispatch to the file, triggered it manually, then went back and added the ruleset successfully. Sorry for all the back and forth, and thanks again for your help!

@bruvv

bruvv commented Aug 9, 2025

Copy link
Copy Markdown
Contributor Author

aah shoot! did not know that. thanks for getting back :)

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