Skip to content

Commit 4d58f9e

Browse files
committed
chore(ui-alerts,console): make documentation preview builds show console logs
INSTUI-4522
1 parent 7450ddc commit 4d58f9e

File tree

6 files changed

+18
-3
lines changed

6 files changed

+18
-3
lines changed

.github/workflows/preview.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@ jobs:
2424
- name: Build docs-app
2525
run: npm run build:docs
2626
if: github.event.action != 'closed'
27+
env:
28+
GITHUB_PULL_REQUEST_PREVIEW: 'true'
2729
- uses: rossjrw/pr-preview-action@v1
2830
with:
2931
source-dir: ./packages/__docs__/__build__

packages/__docs__/webpack.config.mjs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,12 @@ import { globbySync } from 'globby'
3030
import { merge } from 'webpack-merge'
3131
import { processSingleFile } from './lib/build-docs.mjs'
3232
import resolve from './resolve.mjs'
33+
import webpack from 'webpack'
34+
import TerserPlugin from 'terser-webpack-plugin'
3335

3436
const ENV = process.env.NODE_ENV || 'production'
3537
const DEBUG = process.env.DEBUG || ENV === 'development'
38+
const GITHUB_PULL_REQUEST_PREVIEW = process.env.GITHUB_PULL_REQUEST_PREVIEW || 'false'
3639

3740
const outputPath = resolvePath(import.meta.dirname, '__build__')
3841
const resolveAliases = DEBUG ? { resolve } : {}
@@ -79,6 +82,9 @@ const config = merge(baseConfig, {
7982
template: './src/index.html',
8083
chunks: ['main'],
8184
}),
85+
new webpack.DefinePlugin({
86+
'process.env.GITHUB_PULL_REQUEST_PREVIEW': JSON.stringify(GITHUB_PULL_REQUEST_PREVIEW),
87+
}),
8288
],
8389
optimization: {
8490
usedExports: true,

packages/console/src/console.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,11 @@ function logMessage(
5555
message: string,
5656
...args: unknown[]
5757
) {
58-
if (process.env.NODE_ENV !== 'production' && !condition) {
58+
if (
59+
(process.env.GITHUB_PULL_REQUEST_PREVIEW ||
60+
process.env.NODE_ENV !== 'production') &&
61+
!condition
62+
) {
5963
if (typeof console[level] === 'function') {
6064
const renderStack = withRenderStack ? getRenderStack() : ''
6165
//@ts-expect-error level can be 'constructor' which is not callable

packages/ui-alerts/babel.config.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ module.exports = {
2929
{
3030
coverage: Boolean(process.env.COVERAGE),
3131
esModules: Boolean(process.env.ES_MODULES),
32-
removeConsole: process.env.NODE_ENV === 'production',
32+
removeConsole: false,
3333
transformImports: Boolean(process.env.TRANSFORM_IMPORTS)
3434
}
3535
]

packages/ui-alerts/src/Alert/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ const Example = () => {
103103
}
104104
])
105105
setcount(count + 1)
106-
}
106+
}
107107

108108
const closeAlert = (key) =>
109109
setAlerts(alerts.filter((alert) => alert.key !== key))

packages/ui-alerts/src/Alert/index.tsx

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -307,6 +307,9 @@ class Alert extends Component<AlertProps, AlertState> {
307307
}
308308

309309
render() {
310+
// eslint-disable-next-line no-console
311+
console.log('Hello from Alert')
312+
console.warn('Warning from Alert')
310313
const liveRegion = this.getLiveRegion()
311314
const screenReaderContent = liveRegion
312315
? this.createScreenReaderPortal(liveRegion)

0 commit comments

Comments
 (0)