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

[WIP] Default Branches: Permit repos to have a default branch #68

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tombettany
Copy link
Member

https://trello.com/c/MAJSh2Bd/219-3-make-a-rc-package-build-script-or-modify-dr-so-dr-update-can-work-from-a-specific-branch

Currently, the way of determining which branch should be used, should
the branch CLI option not be provided, is to use the currently checked
out branch. This makes it difficult to specify a policy for a suite, or
even have a different branch for each suite.

Add the beginnings of an approach to permit default branches to be
specified by providing an option in the suite definitions which can be
overridden by the package specific options.

@pazdera I'm looking for comments here as I'm not certain that I've gone
the correct route. Also, I had some troubles deciding what the default
behaviour should be so that we maintain backwards compatibility with
repos which do not know about the setting.

The configuration paths were hard-coded every time they were used which
is prone to misconfiguration when changing the paths and reduces
readability for locating these files. Move them into class variables for
reuse elsewhere.
Currently, the way of determining which branch should be used, should
the branch CLI option not be provided, is to use the currently checked
out branch. This makes it difficult to specify a policy for a suite, or
even have a different branch for each suite.

Add the beginnings of an approach to permit default branches to be
specified by providing an option in the suite definitions which can be
overridden by the package specific options.
@Ealdwulf
Copy link
Contributor

Any thoughts on this @pazdera ?

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