LINUX.ORG.RU
ФорумTalks

Про LOR-API и реквесты которые никто не принимает.

 ,


0

1

Вот я взял и посмотрел на «невинно не принятый» пулл:

https://github.com/maxcom/lorsource/pull/642/files

From 03b8cfd31484bb95cfb600a77755b7794c928694 Mon Sep 17 00:00:00 2001
From: 
Date: Tue, 2 Jun 2015 12:39:50 +0300
Subject: [PATCH] Add "/api" prefixes

---
 src/main/java/ru/org/linux/user/UserEventApiController.java | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/main/java/ru/org/linux/user/UserEventApiController.java b/src/main/java/ru/org/linux/user/UserEventApiController.java
index a15c68e..a9291cf 100644
--- a/src/main/java/ru/org/linux/user/UserEventApiController.java
+++ b/src/main/java/ru/org/linux/user/UserEventApiController.java
@@ -35,7 +35,7 @@
   private UserEventService userEventService;
 
   @ResponseBody
-  @RequestMapping(value = "/notifications-count", method= RequestMethod.GET)
+  @RequestMapping(value = "/api/notifications-count", method= RequestMethod.GET)
   public int getEventsCount(HttpServletRequest request, HttpServletResponse response) throws Exception {
     Template tmpl = Template.getTemplate(request);
     if (!tmpl.isSessionAuthorized()) {
@@ -47,7 +47,7 @@ public int getEventsCount(HttpServletRequest request, HttpServletResponse respon
     return tmpl.getCurrentUser().getUnreadEvents();
   }
 
-  @RequestMapping(value="/notifications-reset", method = RequestMethod.POST)
+  @RequestMapping(value="/api/notifications-reset", method = RequestMethod.POST)
   @ResponseBody
   public String resetNotifications(
     HttpServletRequest request,
@@ -66,7 +66,7 @@ public String resetNotifications(
   }
 
   @ResponseBody
-  @RequestMapping(value = "/yandex-tableau", method = RequestMethod.GET, produces={"application/json"})
+  @RequestMapping(value = "/api/yandex-tableau", method = RequestMethod.GET, produces={"application/json"})
   public Map<String, Integer> getYandexWidget(HttpServletRequest request, HttpServletResponse response) throws Exception {
     Template tmpl = Template.getTemplate(request);
     if (!tmpl.isSessionAuthorized()) {
@@ -78,7 +78,7 @@ public String resetNotifications(
     }
   }
 
-  @RequestMapping(value = "/notifications-list", method = RequestMethod.GET, produces = "application/json; charset=UTF-8")
+  @RequestMapping(value = "/api/notifications-list", method = RequestMethod.GET, produces = "application/json; charset=UTF-8")
   @ResponseBody
   public Map<String, Object> listNotifications(
           @RequestParam(value = "filter", required = false) String filterAction,

Для тех кто не понял, скажу, коммит ломает обратную совместимость с имеющимися кодом, причем «просто так».

Сломано просто так, блиан! На адекватное работе за такое эльфийское поведение делают предупредительную прочистку мозгов, если не помогло то увольняют.

С подобным я встречался в своей практике, только тогда мне бедняжко не смогло мотивировать свое поведение, может тут на лоре есть кто-то кто просветит?

Deleted
Ответ на: комментарий от Flame4All

Я думаю в git урлы поправят другими коммитами позже

Я надеюсь, что ты не участвуешь в разработке хоть какого-то софта, которым я так или иначе пользуюсь.

i-rinat ★★★★★
()
Ответ на: комментарий от i-rinat

Ну а как ты представляешь разработку? То что в разрабатываемой ветке что-то ломают в процессе, это нормально. Я не в курсе, есть ли у лоровского движка стабильная ветка, если нет, значит у гневно ещё не было официального релиза.

Flame4All
()
Ответ на: комментарий от i-rinat

судя по коментам в треде, не он - так аналогов полно, хорошо руководители проектов, как правило, ребята опытные и вычищают это из команды

ну а если в проекте все такие то это хорошо заметно по регулярным факапам

Deleted
()
Ответ на: комментарий от Flame4All

То что в разрабатываемой ветке что-то ломают в процессе, это нормально.

Пока эта ветка у тебя на локальной машине или в личном репозитории, поступай как хочешь. Но к моменту pull-request'а всё нужно причесать. Если возможно, код нужно разбить на отдельные патчи, решающие части задачи. При этом каждое промежуточное состояние должно собираться и работать. Иногда для этого нужно не только разбивать один патч на несколько, но и сливать несколько патчей в один.

Просто представь себе, что тебе нужно найти бисекцией патч, из-за которого появилась регрессия. Создавай коммиты так, чтобы это было легко.

Твои патчи будет просматривать человек, которому твои изменения не очень-то и нужны. Уж постарайся сделать так, чтобы они его не бесили.

Я не в курсе, есть ли у лоровского движка стабильная ветка, если нет, значит у гневно ещё не было официального релиза.

Ткнуть на ссылку в заглавном посте, открыть главную страницу репозитория, ткнуть в список веток и поглядеть. Времени занимает меньше, чем написать фразу о том, что ты не в курсе.

Ветка master — и есть стабильная ветка.

i-rinat ★★★★★
()
Ответ на: комментарий от i-rinat

Пока эта ветка у тебя на локальной машине или в личном репозитории, поступай как хочешь.

По ссылке и есть личный репозиторий.

Ветка master — и есть стабильная ветка.

Там об этом ни слова. Т.к. там нет разделения на stable и devel, очевидно что master это devel.

Flame4All
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.