Skip to content

Chore/Project config updates #778

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from
Open

Conversation

dcblundell
Copy link
Contributor

@dcblundell dcblundell commented Aug 20, 2025

  • Adds various bits of metadata to package.json
  • Updated tsconfig.json based on general recommendations
  • Updated shared vscode workspace config
  • Align js target version between tsconfig and Vite

@dcblundell dcblundell marked this pull request as ready for review August 20, 2025 17:40
vite.config.ts Outdated

// https://vitejs.dev/config/
export default defineConfig(({ command }) => {
return {
build: {
outDir: './backend/ttnn_visualizer/static/',
emptyOutDir: true,
target: compilerOptions.target,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure about this bit, but it does keep it consistent.

@dcblundell dcblundell requested a review from Copilot August 20, 2025 17:47
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Updates project configuration files to improve development tooling and standardize metadata. This includes adding package metadata, aligning TypeScript/Vite configurations, and enhancing VS Code workspace settings.

  • Added repository, author, and license metadata to package.json and reorganized its structure
  • Aligned JavaScript target version between tsconfig.json and Vite configuration
  • Enhanced VS Code workspace with additional linting, formatting, and extension recommendations

Reviewed Changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.

File Description
package.json Added metadata fields and reorganized structure, moved license/author to proper location
vite.config.ts Imported tsconfig and aligned build target with TypeScript compiler options
ttnn-visualizer.code-workspace Enhanced VS Code settings with additional linting rules and extension recommendations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@dcblundell dcblundell changed the title Chore/Project config updates chore/project config updates Aug 21, 2025
@dcblundell dcblundell changed the title chore/project config updates Chore/Project config updates Aug 21, 2025
@aidemsined
Copy link
Contributor

please add package.log to gitignore

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants