search

elixir-cloud-aai / krini

Run & manage life science analyses
https://krini.rahtiapp.fi/
Apache License 2.0
4 stars 6 forks source link

chore: migrate to TS #37

Open dhararaghu777 opened 1 month ago

dhararaghu777 commented 1 month ago

IMPORTANT: Please create an issue before filing a pull request! Changes need to be discussed before proceeding. Please read the contribution guidelines.

Details

Please provide enough information so that others can review your pull request. Give a brief summary of the motivation. Refer to the corresponding issue/s with #XXXX for more information.

Testing

Write the appropriate unit and integration tests, if applicable. Make sure these and all other tests pass.

Documentation

Please document your changes and test cases in the appropriate places, if applicable.

Style

Make sure your changes adhere to the coding/documentation style used throughout the project.

Closing issues

If your changes fix any issue/s, put closes #XXXX in your comment to auto-close it/them.

Summary by Sourcery

Migrate the project to a new setup using React, TypeScript, and Vite, replacing the existing Krini application. Update the ESLint configuration to support TypeScript and React, and revise the README to reflect these changes. Remove obsolete components and configuration files related to the previous setup.

New Features:

Enhancements:

Documentation:

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
krini ❌ Failed (Inspect) Oct 12, 2024 0:06am
sourcery-ai[bot] commented 1 month ago

Reviewer's Guide by Sourcery

This pull request implements a significant migration from JavaScript to TypeScript, along with a shift from Create React App to Vite as the build tool. The changes include updating the project structure, configuration files, and core application files to support TypeScript and Vite. The PR also introduces new styling approaches and removes several components, indicating a major refactoring of the application architecture.

File-Level Changes

Files Changes
tsconfig.app.json
tsconfig.node.json
tsconfig.json
src/App.tsx
src/vite-env.d.ts		 Migrated the project from JavaScript to TypeScript, including the addition of TypeScript configuration files and updating existing JavaScript files to TypeScript
vite.config.ts
index.html		 Switched from Create React App to Vite as the build tool, including the addition of Vite configuration
eslint.config.js Updated ESLint configuration to support TypeScript and React
src/index.css
src/App.css		 Replaced Tailwind CSS with vanilla CSS, including new styles for the root element and basic components
src/components/RunWorkflow.js
src/components/Layout.js
src/components/ManageWorkflows.js
src/components/Navbar.js
src/components/Workflow.js
src/components/Footer.js
src/components/Landing.js
src/components/Content.js
src/components/CookieConsent.js
src/components/Privacy.js
src/components/About.js
src/components/Login.js
src/components/Register.js
src/components/TaskRuns.js
src/components/TaskCreate.js
src/components/Profile.js		 Removed multiple React components, indicating a significant change in the application structure
README.md Updated README.md to reflect the new project setup with React, TypeScript, and Vite
.eslintrc.json
tailwind.config.js
postcss.config.js
cypress.config.js
config.js		 Removed configuration files related to the previous setup (Create React App, Tailwind, etc.)
Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - Continue your discussion with Sourcery by replying directly to review comments. - You can change your review settings at any time by accessing your [dashboard](https://app.sourcery.ai): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.