fix: disable pyroscope logo (#890)

* fix: disable pyroscope logo until we figure out a nice watermark
grafana · Feb 24, 2022 · 0477cff · 0477cff
1 parent bb593f0
commit 0477cff
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/packages/pyroscope-flamegraph/package.json b/packages/pyroscope-flamegraph/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@pyroscope/flamegraph",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "main": "dist/index.node.js",
   "browser": "dist/index.js",
   "license": "Apache-2.0",

diff --git a/packages/pyroscope-flamegraph/src/index.spec.tsx b/packages/pyroscope-flamegraph/src/index.spec.tsx
@@ -1,6 +1,5 @@
 import React from 'react';
 import { render, screen } from '@testing-library/react';
-// import { Units } from '@utils/format';
 import { FlamegraphRenderer } from './index';
 
 const SimpleTree = {
@@ -31,7 +30,7 @@ const SimpleTree = {
   spyName: 'gospy',
 };
 
-describe('Pyroscope Library', () => {
+describe.skip('Pyroscope Library', () => {
   it('should not be possible to override the pyroscope logo using props', () => {
     render(
       <FlamegraphRenderer

diff --git a/packages/pyroscope-flamegraph/src/index.tsx b/packages/pyroscope-flamegraph/src/index.tsx
@@ -5,7 +5,8 @@ import FlameGraphRenderer from './FlameGraph/FlameGraphRenderer';
 import Flamegraph from './FlameGraph/FlameGraphComponent/Flamegraph';
 
 const overrideProps = {
-  showPyroscopeLogo: !process.env.PYROSCOPE_HIDE_LOGO as any, // this is injected by webpack
+  //  showPyroscopeLogo: !process.env.PYROSCOPE_HIDE_LOGO as any, // this is injected by webpack
+  showPyroscopeLogo: false,
 };
 
 // TODO: type props

diff --git a/packages/pyroscope-panel-plugin/package.json b/packages/pyroscope-panel-plugin/package.json
@@ -7,6 +7,6 @@
     "dev": "webpack --config ../../scripts/webpack/webpack.panel.ts --watch"
   },
   "dependencies": {
-    "@pyroscope/flamegraph": "^0.0.3"
+    "@pyroscope/flamegraph": "*"
   }
 }