Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode

Поиск
Список
Период
Сортировка
От Surinder Kumar
Тема Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode
Дата
Msg-id CAM5-9D_pxSWe0XZHHsTyznGacc9izcgS6RG0mA=0jXbrzVG5pQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode  (Dave Page <dpage@pgadmin.org>)
Ответы Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers
Hi

On Thu, Sep 7, 2017 at 8:48 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Thu, Sep 7, 2017 at 7:28 AM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi,

When we run Webpack in production mode, it performs optimization on code while in development we don't optimize generated JS and CSS bundles as dev mode is for developer use.

So we should run Webpack bundle in production mode when we are generating bundles for release mode.

Don't we also need a bundle target for dev mode? The patch changes "make bundle" to run "yarn run bundle:prod"
​yes I think so.
We can have two Makefiles - one for dev mode and other for production mode.
 

In the second patch:

1) Enabled "sourced maps" in production mode as well which will help in debugging issues.

2) Removed "yarn run linter" script when Webpack runs in production mode because it is for developer only to check if there are any syntax errors in JS modules.

3) Removed redundant script command "yarn run bundle" as "yarn run bundle:dev" does the same thing.

Please find an attached patch.

Thanks,
Surinder




--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: pgAdmin 4 Jenkins
Дата:
Сообщение: Build failed in Jenkins: pgadmin4-master-python26 #432
Следующее
От: Dave Page
Дата:
Сообщение: Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode