[core,storybook] Remove some and prevent it with an eslint config

Author: sim51

CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## Version 5.0.2
+
+### Fixes
+
+- Remove some `console.log` and prevent it with an eslint config
+
+## Version 5.0.1
+
+Due to some issues with lerna when releasing the v5.0.0, I did this version.
+
 ## Version 5.0.0
 
 ### Breaking changes

eslint.config.mjs

@@ -20,6 +20,7 @@ export default tseslint.config(
       ...reactHooks.configs.recommended.rules,
       "@typescript-eslint/no-unused-vars": ["warn", { "argsIgnorePattern": "^_" }],
       "react/react-in-jsx-scope": "off",
+      "no-console": ["error", { allow: ["warn", "error"] }]
     },
   },
 );

package.json

@@ -26,7 +26,7 @@
     "lerna:changed": "lerna changed",
     "lerna:diff": "lerna diff",
     "lerna:ls": "lerna ls",
-    "lerna:publish": "lerna publish --no-private from-package",
+    "lerna:publish": "npm run clean && npm run build && lerna publish --no-private from-package",
     "lerna:version": "lerna version",
     "lint": "eslint",
     "start": "lerna run start --parallel",

packages/core/src/components/SigmaContainer.tsx

@@ -97,7 +97,6 @@ const SigmaContainerComponent = <
         if (graph) {
           sigGraph = typeof graph === 'function' ? new graph() : graph;
         }
-        console.log('new sigma instance', sigmaSettings);
         let prevCameraState: CameraState | null = null;
         if (prev) {
           prevCameraState = prev.getCamera().getState();

packages/core/src/hooks/useCamera.ts

@@ -65,7 +65,7 @@ export function useCamera(options?: CameraOptions): {
     (nodeKey: string, options?: Partial<AnimateOptions>) => {
       const nodeDisplayData = sigma.getNodeDisplayData(nodeKey);
       if (nodeDisplayData) sigma.getCamera().animate(nodeDisplayData, { ...defaultOptions, ...options });
-      else console.log(`Node ${nodeKey} not found`);
+      else console.warn(`Node ${nodeKey} not found`);
     },
     [sigma, defaultOptions],
   );

packages/storybook/stories/Events.tsx

@@ -11,8 +11,8 @@ const sigmaSettings = { allowInvalidContainer: true };
 const GraphEvents: React.FC = () => {
   const registerEvents = useRegisterEvents();
 
+  /* eslint-disable no-console */
   useEffect(() => {
-    console.log('register events');
     // Register the events
     registerEvents({
       // node events
@@ -58,6 +58,7 @@ const GraphEvents: React.FC = () => {
       updated: (event) => console.log('updated', event.x, event.y, event.angle, event.ratio),
     });
   }, [registerEvents]);
+  /* eslint-enable no-console */
 
   return null;
 };

packages/storybook/stories/GraphSearch.tsx

@@ -30,7 +30,6 @@ export const GraphSearchDemo: FC<{ style?: CSSProperties }> = ({ style }) => {
   const [focusNode, setFocusNode] = useState<string | null>(null);
 
   const onFocus = useCallback((value: GraphSearchOption | null) => {
-    console.log('onfocus', value);
     if (value === null) setFocusNode(null);
     else if (value.type === 'nodes') setFocusNode(value.id);
   }, []);

packages/storybook/stories/LayoutCircular.tsx

@@ -39,6 +39,7 @@ const RandomCircleGraph: FC = () => {
     // Apply the layout
     assign();
     // Display in the console the node's position for the layout
+    // eslint-disable-next-line no-console
     console.log(positions());
   }, [assign, loadGraph, faker.datatype, faker, randomColor, positions]);