Skip to content
This repository has been archived by the owner on Sep 9, 2024. It is now read-only.

The era of Deno begins #364

Merged
merged 16 commits into from
May 8, 2023
Merged

The era of Deno begins #364

merged 16 commits into from
May 8, 2023

Conversation

kachick
Copy link
Owner

@kachick kachick commented May 8, 2023

Import and closes #208

Import and closes #208
@kachick kachick marked this pull request as draft May 8, 2023 04:34
@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2023

Visit the preview URL for this PR (updated for commit 98b7310):

https://color-json-terminal-and-vscode--pr364-deno-retry2-2t8f8ixv.web.app

(expires Mon, 15 May 2023 10:50:45 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 59eb6df3715ec61e9434b8c55d6f326add5e081a

- selector: TSEnumDeclaration
message: Don't declare enums.
prettier/prettier: 'off'
no-mixed-operators: error
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Many options does not exists on deno lint, but brazing fast ⚡ 🙄

"printWidth": 120,
"singleQuote": true
},
"includes": ["**/*.{ts,tsx,js,jsx,cjs,mjs,json,md,yml,html}"],
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would use some formatter for YAML 🤔

import { useState } from 'react';
import './App.css';
import { safeConvertWindowsTerminalToVSCode } from './converter';
import React, { useState } from 'https://esm.sh/[email protected]';
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without React, jsx/tsx does not work in vscode

import App from './App';
import './index.css';
import { assertIsDefined } from './typeguards';
/// <reference lib="dom"/>
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For document.getElementById

import { StrictMode } from 'react';
import { createRoot } from 'react-dom/client';
import App from './App';
import './index.css';
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Importing CSS is not supported in deno typecheck. It was the big blocker 😢

denoland/deno#11961


// https://vitejs.dev/config/
export default defineConfig({
plugins: [react()],
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When deno check, this is not callable 🤔

@kachick kachick changed the title Use deno again The era of Deno begins May 8, 2023
@kachick kachick marked this pull request as ready for review May 8, 2023 10:58
@kachick kachick merged commit 05114d6 into main May 8, 2023
@kachick kachick deleted the deno-retry2 branch May 8, 2023 10:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant