Skip to content

Adding dependencies for python and other project types #15

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.

Sign up for GitHub

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
Ahmed-Dghaies wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Adding dependencies for python and other project types #15

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion frontend/package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 2 additions & 1 deletion frontend/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@
"react-dom": "^18.3.1",
"react-router-dom": "^7.1.1",
"react-tooltip": "^5.28.0",
"tailwind-merge": "^3.2.0"
"tailwind-merge": "^3.2.0",
"toml": "^3.0.0"
},
"devDependencies": {
"@eslint/js": "^9.17.0",
Expand Down
2 changes: 1 addition & 1 deletion frontend/src/features/repositories/components/Dependencies/Dependencies.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ const Dependencies = ({ repositoryDetails }: { repositoryDetails: RepositoryInfo
className="h-full overflow-auto flex flex-col gap-3 w-full md:w-2/3 mb-3 overflow-x-hidden pr-1 min-h-96"
>
{dependenciesCheckIsAvailable === false ? (
<UnderConstruction content="For the time being, this feature is only available for node repositories" />
<UnderConstruction content="For the time being, this feature is only available for Node, Python or PHP repositories" />

@coderabbitai coderabbitai Bot Jun 1, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Update message to reflect all supported project types.

The message only mentions "Node, Python or PHP repositories" but the ProjectType definition includes additional languages like "rust", "go", and "c++". This inconsistency could confuse users about what's actually supported.

Consider updating the message to include all supported types:

-<UnderConstruction content="For the time being, this feature is only available for Node, Python or PHP repositories" />
+<UnderConstruction content="For the time being, this feature is only available for Node, Python, PHP, Rust, Go, and C++ repositories" />

Alternatively, use a more generic message to avoid maintaining a list:

-<UnderConstruction content="For the time being, this feature is only available for Node, Python or PHP repositories" />
+<UnderConstruction content="This feature is currently available for supported project types only" />
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<UnderConstruction content="For the time being, this feature is only available for Node, Python or PHP repositories" />
<UnderConstruction content="For the time being, this feature is only available for Node, Python, PHP, Rust, Go, and C++ repositories" />
🤖 Prompt for AI Agents 
In frontend/src/features/repositories/components/Dependencies/Dependencies.tsx
at line 36, the message in the UnderConstruction component only lists Node,
Python, and PHP as supported project types, but the ProjectType definition
includes additional languages like Rust, Go, and C++. Update the message to
either list all supported project types accurately or replace it with a more
generic statement that covers all supported languages without enumerating them,
ensuring consistency and avoiding confusion.

) : (
dependenciesList.map((dependency) => (
<Dependency packageInfo={dependency} key={dependency.name} />
Expand Down
Loading