8000 chore: update git to only pull master branch for deployment tests by qdzlug · Pull Request #164 · nginxinc/kic-reference-architectures · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Oct 8, 2025. It is now read-only.

Conversation

qdzlug
Copy link
Contributor
@qdzlug qdzlug commented Jun 16, 2022

Proposed changes

The existing Jenkins configuration tried to build all the branches from the repo, but this has proven to be a bit problematic for our use case.

This change restricts the build to */master for now - we can look into a reconfig later to make this part of the merge process, but for now this fixes the immediate concern. See #163 for potential concerns moving forward.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have written my commit messages in the Conventional Commits format.
  • I have read the CONTRIBUTING doc
  • I have added tests (when possible) that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto master
  • I will ensure my PR is targeting the master branch and pulling from my branch from my own fork

@qdzlug qdzlug mentioned this pull request Jun 16, 2022
Copy link
Contributor
@4141done 4141done left a comment

Choose a reason for hiding this comment

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

👍 🐳 👍

@qdzlug qdzlug merged commit bc0b093 into nginxinc:master Jun 20, 2022
@qdzlug qdzlug deleted the jenkins-branch branch June 20, 2022 15:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0