[PR #297] [MERGED] devcontainer v1 #3290

Closed
opened 2026-02-22 18:33:28 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Mintplex-Labs/anything-llm/pull/297
Author: @franzbischoff
Created: 10/26/2023
Status: Merged
Merged: 1/8/2024
Merged by: @timothycarambat

Base: masterHead: feature/devcont


📝 Commits (10+)

📊 Changes

24 files changed (+2309 additions, -1016 deletions)

View changed files

.devcontainer/README.md (+72 -0)
.devcontainer/devcontainer.json (+211 -0)
📝 .dockerignore (+9 -8)
📝 .editorconfig (+9 -4)
📝 .gitignore (+2 -1)
.prettierignore (+12 -0)
.prettierrc (+38 -0)
.vscode/launch.json (+74 -0)
📝 .vscode/settings.json (+2 -1)
.vscode/tasks.json (+94 -0)
📝 collector/.env.example (+1 -1)
eslint.config.js (+90 -0)
📝 frontend/.env.example (+1 -0)
frontend/.eslintrc.cjs (+0 -20)
📝 frontend/.gitignore (+2 -0)
📝 frontend/package.json (+16 -7)
📝 frontend/tailwind.config.js (+65 -31)
📝 frontend/vite.config.js (+23 -21)
📝 frontend/yarn.lock (+1528 -918)
📝 package.json (+3 -2)

...and 4 more files

📄 Description

This PR does the following:

  • Added configuration for creating a devcontainer with the environment ready for starting server:dev and frontend:dev
  • Changes the location of eslint config in order to be global. In fact, Eslint is mostly useful when employed with the IDE extension. The package lint script uses prettier, not eslint.
  • Creates a prettier config file and ignore file, so both CLI and IDE extensions have the same formatting rules.
  • Tailwind is now integrated into the IDE.

Adjustments:

  • settings.json needs the eslint config, so VSCode doesn't complain about the old format.
  • package-lock.json is removed in favor of yarn.lock. The project uses yarn, not npm.
  • Some adjustments were made on all "package.json" files.
  • Added SERVER_ADDRESS in server/.env.example to fix a bug when using WSL.

Final words:

These standardizations of code format don't require to use of the devcontainer. One can still run the server and frontend and configure the extensions by hand in the system environment. Nevertheless, the devcontainer is a safer and quicker way to start developing.

I've tested in several ways and it is working. But who knows... 🤞


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Mintplex-Labs/anything-llm/pull/297 **Author:** [@franzbischoff](https://github.com/franzbischoff) **Created:** 10/26/2023 **Status:** ✅ Merged **Merged:** 1/8/2024 **Merged by:** [@timothycarambat](https://github.com/timothycarambat) **Base:** `master` ← **Head:** `feature/devcont` --- ### 📝 Commits (10+) - [`b4993bc`](https://github.com/Mintplex-Labs/anything-llm/commit/b4993bc7b18c33ed728837e054f9a5aa8b5b27f6) devcontainer v1 - [`8fd0946`](https://github.com/Mintplex-Labs/anything-llm/commit/8fd09460c3fa1041607598d0a69bb0b9a7021b68) tweaks - [`fa6f135`](https://github.com/Mintplex-Labs/anything-llm/commit/fa6f13570f3b749530e1422ea35f732af26d22eb) tweaks 2 - [`a1cd687`](https://github.com/Mintplex-Labs/anything-llm/commit/a1cd68770d3df4e19ddd86e4ab0d943c06e477c2) rebase - [`417b3ee`](https://github.com/Mintplex-Labs/anything-llm/commit/417b3ee4f1172f7ac23183bde32551c1559dfb82) Update AWS deployment - [`e483c7a`](https://github.com/Mintplex-Labs/anything-llm/commit/e483c7a75c94d025e181c21cf13e88cc189c6a03) mbox parsing improvements v1 (#308) - [`b73a59a`](https://github.com/Mintplex-Labs/anything-llm/commit/b73a59ab8a18a746bb9eda19fa4d00793a40badf) 282 return relevancy score with similarityresponse (#304) - [`d59058a`](https://github.com/Mintplex-Labs/anything-llm/commit/d59058af73bc6df49ac68531ed14db3b3640b92c) Rebase - [`fe4eeed`](https://github.com/Mintplex-Labs/anything-llm/commit/fe4eeed1024c4c394ba7bda8c9492b584db3a204) fix import path - [`5869d8b`](https://github.com/Mintplex-Labs/anything-llm/commit/5869d8b1cba4880dbf7acfe3d4a924c163918a24) Implement retrieval and use of fine-tune models (#314) ### 📊 Changes **24 files changed** (+2309 additions, -1016 deletions) <details> <summary>View changed files</summary> ➕ `.devcontainer/README.md` (+72 -0) ➕ `.devcontainer/devcontainer.json` (+211 -0) 📝 `.dockerignore` (+9 -8) 📝 `.editorconfig` (+9 -4) 📝 `.gitignore` (+2 -1) ➕ `.prettierignore` (+12 -0) ➕ `.prettierrc` (+38 -0) ➕ `.vscode/launch.json` (+74 -0) 📝 `.vscode/settings.json` (+2 -1) ➕ `.vscode/tasks.json` (+94 -0) 📝 `collector/.env.example` (+1 -1) ➕ `eslint.config.js` (+90 -0) 📝 `frontend/.env.example` (+1 -0) ➖ `frontend/.eslintrc.cjs` (+0 -20) 📝 `frontend/.gitignore` (+2 -0) 📝 `frontend/package.json` (+16 -7) 📝 `frontend/tailwind.config.js` (+65 -31) 📝 `frontend/vite.config.js` (+23 -21) 📝 `frontend/yarn.lock` (+1528 -918) 📝 `package.json` (+3 -2) _...and 4 more files_ </details> ### 📄 Description This PR does the following: - Added configuration for creating a devcontainer with the environment ready for starting server:dev and frontend:dev - Changes the location of eslint config in order to be global. In fact, Eslint is mostly useful when employed with the IDE extension. The package lint script uses prettier, not eslint. - Creates a prettier config file and ignore file, so both CLI and IDE extensions have the same formatting rules. - Tailwind is now integrated into the IDE. Adjustments: - settings.json needs the eslint config, so VSCode doesn't complain about the old format. - package-lock.json is removed in favor of yarn.lock. The project uses yarn, not npm. - Some adjustments were made on all "package.json" files. - Added SERVER_ADDRESS in server/.env.example to fix a bug when using WSL. Final words: These standardizations of code format don't require to use of the devcontainer. One can still run the server and frontend and configure the extensions by hand in the system environment. Nevertheless, the devcontainer is a safer and quicker way to start developing. I've tested in several ways and it is working. But who knows... 🤞 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-22 18:33:28 -05:00
yindo closed this issue 2026-02-22 18:33:28 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Mintplex-Labs/anything-llm#3290