search

sealcode / sealious

An extensible, declarative node framework
25 stars 2 forks source link

Zmienić nazwę modułu core'owego prometheus-module #24

Closed Delfinek closed 9 years ago

Delfinek commented 9 years ago

Moduł core'owy (ten bin/node_modules/prometheus-modules) ma taką samą nazwę jak katalog w którym trzymamy "funkcyjne" moduły prometheusa. Uważamy, że należy zmienić nazwę modułu core'owego, żeby nie konfudować na np. prometheus-modules-manages.

Trzba pamiętać o zmianie nazy wszędzie, gdzie występuje odniesienie do tego modułu!

simplypixi commented 9 years ago

Może "promethues-core-modules"? od razu wiadomo, że core :D Kuba możesz mi to podpiąć. ;)

kuba-orlik commented 9 years ago

Hmmmm a może prometheus-resource-manager? On Nov 29, 2014 7:26 PM, "Mariusz" notifications@github.com wrote:

Może "promethues-core-modules"? od razu wiadomo, że core :D Kuba możesz mi to podpiąć. ;)

— Reply to this email directly or view it on GitHub https://github.com/groovy354/Prometheus/issues/24#issuecomment-64960629.

simplypixi commented 9 years ago

Już mamy prometheus-resource-manager. ;p Chyba, że sugerujesz, aby połączyć jedno i drugie? :P

kuba-orlik commented 9 years ago

Hmmm ale obydwa robią rożne rzeczy On Nov 29, 2014 7:45 PM, "Mariusz" notifications@github.com wrote:

Już mamy prometheus-resource-manager. ;p Chyba, że sugerujesz, aby połączyć jedno i drugie? :P

— Reply to this email directly or view it on GitHub https://github.com/groovy354/Prometheus/issues/24#issuecomment-64961223.

simplypixi commented 9 years ago

No tak ;p Próbuję zrozumieć chęć użycia takiego samego nazewnictwa dla dwóch różnych funkcjonalności. Mniejsza. Nazwa "prometheus-resource-manager" jest zajęta. ;)

kuba-orlik commented 9 years ago

Haha chciałem napisać "prometheus-module-manager", a nie "prometheus-resource-manager". Wierzcie lub nie, to jest wina autokorekty :smile: Podsumowując - czy uznajecie za dobry pomysł zmianę nazwy core'owego modułu "prometheus-modules" na "prometheus-module-manager"?

adwydman commented 9 years ago

Uznajecie.

kuba-orlik commented 9 years ago

Ok, kto zgłasza sprzeciw niech odezwie się teraz albo zamilknie na wieki ;p

kuba-orlik commented 9 years ago

Postanowione wiec :) prometheus-resource-manager

kuba-orlik commented 9 years ago

Module*

simplypixi commented 9 years ago

Zrobione. Nazwa core'owego "prometheus-modules" została zmieniona na "prometheus-module-manager".