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

TEST: PR test for builds on merge requests #62

Closed
wants to merge 20 commits into from

Conversation

PlaidCat
Copy link
Collaborator

No description provided.

PlaidCat and others added 3 commits January 13, 2025 13:23
THIS IS BREAKING AND NOT INTENDED TO BE SHIPPED
Since we need to make sure all our checks run BEFORE the commit is
pushed to a branch we're swithcing this to `on pull_request_target` to
enable external contibutors.
@PlaidCat PlaidCat requested a review from bmastbergen January 14, 2025 19:04
@PlaidCat PlaidCat closed this Jan 14, 2025
@PlaidCat
Copy link
Collaborator Author

This did not do what I expected

PlaidCat and others added 2 commits January 14, 2025 15:53
Since we need to make sure all our checks run BEFORE the commit is
pushed to a branch we're swithcing this to `on pull_request_target` to
enable external contibutors.
@PlaidCat PlaidCat reopened this Jan 14, 2025
Testing resubmission fix
trying to move from:
merge_commit_sha to HEAD
PlaidCat and others added 12 commits January 14, 2025 18:40
Since we need to make sure external contributors code actually compiles
prior to merging. To get access to the forked repos merge request we
need to switch over our push/pull_request to pull_request_target.  In
addition we're fixing up some Naming Conventions, adding aarch64 to this
branch and fixing the naming so that we can quickly identify if the CI
is for x86_64 or aarch64.
@PlaidCat PlaidCat force-pushed the {jmaple}_ciqlts8_8 branch 3 times, most recently from 105bc2d to ebe2551 Compare January 15, 2025 22:20
@PlaidCat PlaidCat force-pushed the {jmaple}_ciqlts8_8 branch 2 times, most recently from 5a2bd5b to d38c5b9 Compare January 15, 2025 22:27
@PlaidCat
Copy link
Collaborator Author

Closing for now

@PlaidCat PlaidCat closed this Jan 15, 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.

1 participant