Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
2676f41def | ||
|
|
22fd68c5b6 | ||
|
|
04c9a4b449 | ||
|
|
417021b36e | ||
|
|
be245114d5 | ||
|
|
184b135886 | ||
|
|
acc0e8666a | ||
|
|
5806aeab27 | ||
|
|
23eb9df482 | ||
|
|
538c9130dc | ||
|
|
a0d3028904 | ||
|
|
59b9e1c419 | ||
|
|
8793248dbe | ||
|
|
f389118515 | ||
|
|
5a7ed03801 | ||
|
|
5f7db6cf5d |
@@ -1,3 +1,31 @@
|
|||||||
|
## v1.17.1 - 2026-05-20
|
||||||
|
|
||||||
|
- **bugfix**: dont push environment branches since they are already created on remote [[jonathan.nerat](mailto:jonathan.nerat@gmail.com)] (04c9a4b)
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
|
||||||
|
## v1.17.0 - 2026-05-20
|
||||||
|
|
||||||
|
- **feature**: allow merges between environments [[jonathan.nerat](mailto:jonathan.nerat@gmail.com)] (5806aea)
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
|
||||||
|
## v1.16.1 - 2026-05-20
|
||||||
|
|
||||||
|
- **bugfix**: arreglo split de branch para armar correctamente el titulo del PR [[jteran](mailto:jteran@renatre.org.ar)] (a0d3028)
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
|
||||||
|
## v1.16.0 - 2026-05-20
|
||||||
|
|
||||||
|
- **feature**: agrego soporte para especificar tickets en nombre e rama [[jonathan.nerat](mailto:jonathan.nerat@gmail.com)] (f389118)
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
|
||||||
## v1.15.0 - 2026-05-17
|
## v1.15.0 - 2026-05-17
|
||||||
|
|
||||||
- **feature**: include revlist count in version tag when skipping bump [[jonathan.nerat](mailto:jonathan.nerat@gmail.com)] (d275ac0)
|
- **feature**: include revlist count in version tag when skipping bump [[jonathan.nerat](mailto:jonathan.nerat@gmail.com)] (d275ac0)
|
||||||
|
|||||||
@@ -49,3 +49,5 @@ SUPPORTED_REMOTE_APIS = [
|
|||||||
]
|
]
|
||||||
|
|
||||||
REPOSITORY_TOKEN_FILENAME = ".repository-token"
|
REPOSITORY_TOKEN_FILENAME = ".repository-token"
|
||||||
|
|
||||||
|
FLOWCONFIG_FILENAME = ".flowconfig"
|
||||||
|
|||||||
@@ -1,9 +1,10 @@
|
|||||||
from git_flow import (
|
from git_flow import (
|
||||||
BRANCH_TYPES,
|
BRANCH_TYPES,
|
||||||
REPOSITORY_TOKEN_FILENAME,
|
REPOSITORY_TOKEN_FILENAME,
|
||||||
|
FLOWCONFIG_FILENAME,
|
||||||
GitFlowError,
|
GitFlowError,
|
||||||
)
|
)
|
||||||
from git_flow.git import FLOWCONFIG_FILE, Git
|
from git_flow.git import Git
|
||||||
from git_flow.remote.base import RemoteAPI
|
from git_flow.remote.base import RemoteAPI
|
||||||
from git_flow.remote.bitbucket import BitbucketRemoteAPI
|
from git_flow.remote.bitbucket import BitbucketRemoteAPI
|
||||||
from git_flow.remote.github import GithubRemoteAPI
|
from git_flow.remote.github import GithubRemoteAPI
|
||||||
@@ -16,8 +17,8 @@ TYPE_ERROR = 2
|
|||||||
TYPE_INFO = 3
|
TYPE_INFO = 3
|
||||||
|
|
||||||
|
|
||||||
flowconfig = Git.get_config(FLOWCONFIG_FILE) if isfile(FLOWCONFIG_FILE) else {}
|
flowconfig = Git.get_config(FLOWCONFIG_FILENAME) if isfile(FLOWCONFIG_FILENAME) else {}
|
||||||
|
environments = flowconfig["flow.branches"].split(",") if "flow.branches" in flowconfig else []
|
||||||
|
|
||||||
def ensure_initialized():
|
def ensure_initialized():
|
||||||
initialized = flowconfig["flow.initialized"] if flowconfig else None
|
initialized = flowconfig["flow.initialized"] if flowconfig else None
|
||||||
@@ -45,7 +46,6 @@ def ensure_right_branch():
|
|||||||
|
|
||||||
def get_branch_env_and_type(branch: str) -> tuple[str, str]:
|
def get_branch_env_and_type(branch: str) -> tuple[str, str]:
|
||||||
components = branch.split("/")
|
components = branch.split("/")
|
||||||
target_branches = flowconfig["flow.branches"].split(",")
|
|
||||||
|
|
||||||
if len(components) == 2:
|
if len(components) == 2:
|
||||||
if components[0] not in BRANCH_TYPES:
|
if components[0] not in BRANCH_TYPES:
|
||||||
@@ -53,13 +53,13 @@ def get_branch_env_and_type(branch: str) -> tuple[str, str]:
|
|||||||
f"Branch inválida, el tipo '{components[0]}' no es válido."
|
f"Branch inválida, el tipo '{components[0]}' no es válido."
|
||||||
)
|
)
|
||||||
|
|
||||||
return (target_branches[0], components[0])
|
return (environments[0], components[0])
|
||||||
elif len(components) == 4:
|
elif len(components) == 4:
|
||||||
target_branches = target_branches[1:]
|
valid_environments = environments[1:]
|
||||||
|
|
||||||
if (
|
if (
|
||||||
components[0] != "release"
|
components[0] != "release"
|
||||||
or components[1] not in target_branches
|
or components[1] not in valid_environments
|
||||||
or components[2] not in BRANCH_TYPES
|
or components[2] not in BRANCH_TYPES
|
||||||
):
|
):
|
||||||
raise GitFlowError(
|
raise GitFlowError(
|
||||||
@@ -112,3 +112,15 @@ def get_remote_api(token: str) -> RemoteAPI:
|
|||||||
return GithubRemoteAPI(repository, token)
|
return GithubRemoteAPI(repository, token)
|
||||||
else:
|
else:
|
||||||
raise GitFlowError("El host del repositorio remoto es inválido")
|
raise GitFlowError("El host del repositorio remoto es inválido")
|
||||||
|
|
||||||
|
def is_valid_ticket(ticket: str) -> bool:
|
||||||
|
"""Check if the specified string is a valid ticket number (ABC-123)"""
|
||||||
|
components = ticket.split("-")
|
||||||
|
|
||||||
|
if len(components) != 2:
|
||||||
|
return False
|
||||||
|
|
||||||
|
ticket_project = components[0]
|
||||||
|
ticket_number = components[1]
|
||||||
|
|
||||||
|
return ticket_project.isalpha() and ticket_project.isupper() and ticket_number.isnumeric()
|
||||||
@@ -41,7 +41,6 @@ def branch(
|
|||||||
ensure_initialized()
|
ensure_initialized()
|
||||||
|
|
||||||
current_branch = Git.get_current_branch()
|
current_branch = Git.get_current_branch()
|
||||||
environments = flowconfig["flow.branches"].split(",")
|
|
||||||
|
|
||||||
if current_branch in environments:
|
if current_branch in environments:
|
||||||
current_environment = current_branch
|
current_environment = current_branch
|
||||||
|
|||||||
@@ -1,7 +1,7 @@
|
|||||||
import typer
|
import typer
|
||||||
|
|
||||||
from git_flow import GitFlowError
|
from git_flow import FLOWCONFIG_FILENAME, GitFlowError
|
||||||
from git_flow.git import FLOWCONFIG_FILE, Git
|
from git_flow.git import Git
|
||||||
from git_flow.command.base import *
|
from git_flow.command.base import *
|
||||||
|
|
||||||
app = typer.Typer()
|
app = typer.Typer()
|
||||||
@@ -25,8 +25,8 @@ def init():
|
|||||||
if remote:
|
if remote:
|
||||||
flowconfig["flow.remote"] = remote
|
flowconfig["flow.remote"] = remote
|
||||||
|
|
||||||
Git.set_config(flowconfig, FLOWCONFIG_FILE)
|
Git.set_config(flowconfig, FLOWCONFIG_FILENAME)
|
||||||
Git("add", FLOWCONFIG_FILE).exec()
|
Git("add", FLOWCONFIG_FILENAME).exec()
|
||||||
Git("commit", message="feature: initialize git-flow").exec()
|
Git("commit", message="feature: initialize git-flow").exec()
|
||||||
|
|
||||||
_ensure_all_flow_branches_exist(branches, remote)
|
_ensure_all_flow_branches_exist(branches, remote)
|
||||||
@@ -43,7 +43,6 @@ def _ensure_is_repository():
|
|||||||
|
|
||||||
|
|
||||||
def _ensure_not_already_initialized():
|
def _ensure_not_already_initialized():
|
||||||
flowconfig = get_config()
|
|
||||||
if not flowconfig:
|
if not flowconfig:
|
||||||
return
|
return
|
||||||
elif flowconfig.get("flow.initialized") == "true":
|
elif flowconfig.get("flow.initialized") == "true":
|
||||||
|
|||||||
@@ -13,6 +13,21 @@ def merge():
|
|||||||
ensure_initialized()
|
ensure_initialized()
|
||||||
|
|
||||||
branch = ensure_right_branch()
|
branch = ensure_right_branch()
|
||||||
|
|
||||||
|
if branch in environments:
|
||||||
|
if branch == environments[-1]:
|
||||||
|
raise GitFlowError("No se puede hacer un merge del último entorno.")
|
||||||
|
|
||||||
|
panel(
|
||||||
|
"Merge de entornos",
|
||||||
|
"""Está por hacer un merge de 2 entornos que puede implicar muchos cambios en el proyecto.""",
|
||||||
|
)
|
||||||
|
|
||||||
|
if not confirm("¿Desea continuar?"):
|
||||||
|
raise GitFlowError("Ejecución abortada")
|
||||||
|
|
||||||
|
target = environments[environments.index(branch) + 1]
|
||||||
|
else:
|
||||||
target, _ = get_branch_env_and_type(branch)
|
target, _ = get_branch_env_and_type(branch)
|
||||||
|
|
||||||
if "flow.remote" in flowconfig:
|
if "flow.remote" in flowconfig:
|
||||||
@@ -34,6 +49,7 @@ def run_remote(remote: str, token: str, branch: str, target: str):
|
|||||||
show_commits_to_merge(target)
|
show_commits_to_merge(target)
|
||||||
|
|
||||||
if confirm(f"¿Crear PR de '{branch}' a '{target}'?"):
|
if confirm(f"¿Crear PR de '{branch}' a '{target}'?"):
|
||||||
|
if branch not in environments:
|
||||||
Git("push", remote, branch, set_upstream=True).exec(
|
Git("push", remote, branch, set_upstream=True).exec(
|
||||||
print="Subiendo rama al remoto para crear PR"
|
print="Subiendo rama al remoto para crear PR"
|
||||||
)
|
)
|
||||||
@@ -83,21 +99,12 @@ def check_merge_conflicts(target: str):
|
|||||||
def create_pull_request(token: str, branch: str, target: str):
|
def create_pull_request(token: str, branch: str, target: str):
|
||||||
changelog = Changelog()
|
changelog = Changelog()
|
||||||
commits = Git("log", target + "..", format="%s").lines()
|
commits = Git("log", target + "..", format="%s").lines()
|
||||||
|
title = commits[0] if len(commits) == 1 else _get_pr_title_from_branch(branch)
|
||||||
|
|
||||||
if len(commits) == 1:
|
panel("Título del PR", title)
|
||||||
title = commits[0]
|
|
||||||
else:
|
|
||||||
title = branch.replace("/", ": ").replace("-", " ")
|
|
||||||
|
|
||||||
if title.startswith("release: "):
|
if confirm("¿Desea cambiar el título del PR?", False):
|
||||||
title = title[title.index(" ", title.index(" ") + 1) + 1 :]
|
title = prompt("Título del PR")
|
||||||
|
|
||||||
info("Título del PR por defecto: " + title)
|
|
||||||
|
|
||||||
opt_title = prompt("[Opcional] Ingrese otro titulo para el PR")
|
|
||||||
|
|
||||||
if opt_title:
|
|
||||||
title = opt_title
|
|
||||||
|
|
||||||
message = get_remote_api(token).create_pull_request(
|
message = get_remote_api(token).create_pull_request(
|
||||||
branch,
|
branch,
|
||||||
@@ -111,3 +118,26 @@ def create_pull_request(token: str, branch: str, target: str):
|
|||||||
if confirm("¿Desea cambiar a la rama objetivo y bajar los cambios?"):
|
if confirm("¿Desea cambiar a la rama objetivo y bajar los cambios?"):
|
||||||
Git("switch", target).exec(print="Cambiando a rama objetivo")
|
Git("switch", target).exec(print="Cambiando a rama objetivo")
|
||||||
Git("pull").exec(print="Obteniendo cambios")
|
Git("pull").exec(print="Obteniendo cambios")
|
||||||
|
|
||||||
|
def _get_pr_title_from_branch(branch: str) -> str:
|
||||||
|
components = branch.split("/") # <type>/<desc> or release/<env>/<type>/<desc>
|
||||||
|
branch_type = components[-2]
|
||||||
|
branch_desc = components[-1]
|
||||||
|
|
||||||
|
# detect if branch description has ticket as prefix: <desc> = ABC-123-my-branch-name
|
||||||
|
branch_ticket = None
|
||||||
|
first_dash = branch_desc.find("-")
|
||||||
|
second_dash = branch_desc.find("-", first_dash + 1)
|
||||||
|
|
||||||
|
if first_dash >= 0 and second_dash >= 0:
|
||||||
|
maybe_ticket = branch_desc[:second_dash]
|
||||||
|
if is_valid_ticket(maybe_ticket):
|
||||||
|
branch_ticket = maybe_ticket
|
||||||
|
branch_desc = branch_desc[:second_dash+1]
|
||||||
|
|
||||||
|
return " ".join(filter(None, [
|
||||||
|
f"[Pasaje a {components[1]}]" if len(components) == 4 else None,
|
||||||
|
branch_type + ":",
|
||||||
|
branch_ticket,
|
||||||
|
branch_desc.replace("-", " ")
|
||||||
|
]))
|
||||||
@@ -2,6 +2,7 @@ import typer
|
|||||||
from git_flow import BRANCH_TYPES
|
from git_flow import BRANCH_TYPES
|
||||||
from git_flow.command.base import *
|
from git_flow.command.base import *
|
||||||
from git_flow.git import Git
|
from git_flow.git import Git
|
||||||
|
from typing import Optional
|
||||||
|
|
||||||
app = typer.Typer()
|
app = typer.Typer()
|
||||||
|
|
||||||
@@ -12,16 +13,16 @@ def new():
|
|||||||
|
|
||||||
ensure_initialized()
|
ensure_initialized()
|
||||||
branch = ensure_right_branch()
|
branch = ensure_right_branch()
|
||||||
envs = flowconfig["flow.branches"].split(",")
|
|
||||||
|
|
||||||
if branch not in envs:
|
if branch not in environments:
|
||||||
warning("La rama actual no corresponde a un entorno configurado.")
|
warning("La rama actual no corresponde a un entorno configurado.")
|
||||||
|
|
||||||
info("Se creará una nueva rama de trabajo sobre la rama actual.")
|
info("Se creará una nueva rama de trabajo sobre la rama actual.")
|
||||||
info("Debe seleccionar el tipo de cambio a realizar.")
|
info("Debe seleccionar el tipo de cambio a realizar.")
|
||||||
|
|
||||||
branch_type = choice("Tipos de cambio", BRANCH_TYPES)
|
branch_type = choice("Tipos de cambio", BRANCH_TYPES)
|
||||||
new_branch = _get_unique_branch_name(branch_type)
|
ticket = _get_optional_ticket()
|
||||||
|
new_branch = _get_unique_branch_name(branch_type, ticket)
|
||||||
|
|
||||||
if "flow.remote" in flowconfig and Git.get_tracking_branch(branch):
|
if "flow.remote" in flowconfig and Git.get_tracking_branch(branch):
|
||||||
Git("pull").exec(print="Actualizando rama actual")
|
Git("pull").exec(print="Actualizando rama actual")
|
||||||
@@ -29,8 +30,18 @@ def new():
|
|||||||
if confirm(f"¿Crear rama '{new_branch}' sobre la rama actual?"):
|
if confirm(f"¿Crear rama '{new_branch}' sobre la rama actual?"):
|
||||||
Git("switch", new_branch, create=True).exec(print="Creando nueva rama")
|
Git("switch", new_branch, create=True).exec(print="Creando nueva rama")
|
||||||
|
|
||||||
|
def _get_optional_ticket() -> Optional[str]:
|
||||||
|
ticket = prompt("[Opcional] Ticket que respalda el cambio (en formato ABC-123)")
|
||||||
|
|
||||||
def _get_unique_branch_name(branch_type: str) -> str:
|
if not ticket:
|
||||||
|
return None
|
||||||
|
if not is_valid_ticket(ticket):
|
||||||
|
warning("Formato de ticket inválido, ignorando.")
|
||||||
|
return None
|
||||||
|
|
||||||
|
return ticket
|
||||||
|
|
||||||
|
def _get_unique_branch_name(branch_type: str, ticket: Optional[str]) -> str:
|
||||||
info(f"Ingrese las palabras clave que describan el cambio de tipo '{branch_type}'.")
|
info(f"Ingrese las palabras clave que describan el cambio de tipo '{branch_type}'.")
|
||||||
branch = None
|
branch = None
|
||||||
branches = Git.get_branches()
|
branches = Git.get_branches()
|
||||||
@@ -42,7 +53,7 @@ def _get_unique_branch_name(branch_type: str) -> str:
|
|||||||
)
|
)
|
||||||
|
|
||||||
keywords = filter(lambda k: len(k) > 0, keywords.split(" "))
|
keywords = filter(lambda k: len(k) > 0, keywords.split(" "))
|
||||||
branch = branch_type + "/" + "-".join(keywords)
|
branch = branch_type + "/" + (ticket + "-" if ticket else "") + "-".join(keywords)
|
||||||
|
|
||||||
if branch in branches:
|
if branch in branches:
|
||||||
error(
|
error(
|
||||||
|
|||||||
@@ -13,10 +13,9 @@ def release(group: Optional[str] = None):
|
|||||||
ensure_initialized()
|
ensure_initialized()
|
||||||
|
|
||||||
branch = ensure_right_branch()
|
branch = ensure_right_branch()
|
||||||
envs = flowconfig["flow.branches"].split(",")
|
|
||||||
env, _ = get_branch_env_and_type(branch)
|
env, _ = get_branch_env_and_type(branch)
|
||||||
|
|
||||||
if env == envs[-1]:
|
if env == environments[-1]:
|
||||||
raise GitFlowError(
|
raise GitFlowError(
|
||||||
"No se puede hacer release de una rama en el ultimo entorno."
|
"No se puede hacer release de una rama en el ultimo entorno."
|
||||||
)
|
)
|
||||||
@@ -25,7 +24,7 @@ def release(group: Optional[str] = None):
|
|||||||
|
|
||||||
ensure_clean_worktree(has_remote)
|
ensure_clean_worktree(has_remote)
|
||||||
|
|
||||||
next_env = envs[envs.index(env) + 1]
|
next_env = environments[environments.index(env) + 1]
|
||||||
next_branch = (
|
next_branch = (
|
||||||
branch.replace(f"/{env}/", f"/{next_env}/")
|
branch.replace(f"/{env}/", f"/{next_env}/")
|
||||||
if branch.startswith("release/")
|
if branch.startswith("release/")
|
||||||
|
|||||||
@@ -20,10 +20,9 @@ def tag(token: Optional[str] = None):
|
|||||||
"""Crea un nuevo tag para el último merge."""
|
"""Crea un nuevo tag para el último merge."""
|
||||||
ensure_initialized()
|
ensure_initialized()
|
||||||
|
|
||||||
envs = flowconfig["flow.branches"].split(",")
|
|
||||||
target = Git.get_current_branch()
|
target = Git.get_current_branch()
|
||||||
|
|
||||||
if target not in envs:
|
if target not in environments:
|
||||||
raise GitFlowError("Solo se pueden taggear commits en ramas principales.")
|
raise GitFlowError("Solo se pueden taggear commits en ramas principales.")
|
||||||
|
|
||||||
branch = Git(
|
branch = Git(
|
||||||
|
|||||||
+2
-7
@@ -1,13 +1,8 @@
|
|||||||
import subprocess
|
import subprocess
|
||||||
|
|
||||||
|
from git_flow import FLOWCONFIG_FILENAME
|
||||||
from git_flow.io import *
|
from git_flow.io import *
|
||||||
|
|
||||||
FLOWCONFIG_FILE = ".flowconfig"
|
|
||||||
|
|
||||||
CHANGELOG_FILE = "CHANGELOG.md"
|
|
||||||
|
|
||||||
BUMP_VERSION = "chore: bump version and update CHANGELOG.md [skip ci]"
|
|
||||||
|
|
||||||
|
|
||||||
class Git:
|
class Git:
|
||||||
class Status:
|
class Status:
|
||||||
@@ -42,7 +37,7 @@ class Git:
|
|||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def flow_config(*args, **kwargs):
|
def flow_config(*args, **kwargs):
|
||||||
return Git("config", *args, file=FLOWCONFIG_FILE, **kwargs)
|
return Git("config", *args, file=FLOWCONFIG_FILENAME, **kwargs)
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def get_current_branch():
|
def get_current_branch():
|
||||||
|
|||||||
Reference in New Issue
Block a user