Skip to content

Conversation

@KeyMatrix
Copy link
Owner

No description provided.

Signed-off-by: KeyMatrix <mus.musayev@gmail.com>
Copy link

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.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • .github/workflows/requirements.txt: Language not supported

@KeyMatrix KeyMatrix requested a review from Copilot April 27, 2025 07:57
Copy link

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.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • .github/workflows/requirements.txt: Language not supported

Copy link

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.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • .github/workflows/requirements.txt: Language not supported

@KeyMatrix KeyMatrix linked an issue May 12, 2025 that may be closed by this pull request
@KeyMatrix KeyMatrix requested a review from Copilot May 13, 2025 02:04
Copy link

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

This PR adds a new environment specification file to configure the project's dependencies.

  • Introduces a new requirements file for the workflow under .github/workflows.
  • Specifies both conda and pip dependencies including Python, flask, websocket-client, pytest, and flake8.
Comments suppressed due to low confidence (1)

.github/workflows/requirements.txt:1

  • The file is named 'requirements.txt' which typically implies a pip requirements file, but the content appears to define a conda environment. Consider renaming it to 'environment.yml' to clarify its purpose.
name: treeom-env

@KeyMatrix KeyMatrix added bug Something isn't working documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers invalid This doesn't seem right question Further information is requested wontfix This will not be worked on Babat labels May 13, 2025
@KeyMatrix KeyMatrix requested a review from Copilot May 13, 2025 02:05
Copy link

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

This PR introduces a Conda environment definition for CI workflows, specifying channels and key dependencies.

  • Adds a new Conda environment file (requirements.txt) under .github/workflows
  • Pins Python, Flask, WebSocket client, pytest, and flake8 versions
  • Includes a nested pip install step referencing the main requirements.txt
Comments suppressed due to low confidence (1)

.github/workflows/requirements.txt:1

  • This file uses YAML syntax for a Conda environment but is named requirements.txt; consider renaming it to environment.yml to clearly indicate its purpose.
name: treeom-env

@@ -0,0 +1,13 @@
name: treeom-env
Copy link

Copilot AI May 13, 2025

Choose a reason for hiding this comment

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

Files in .github/workflows are reserved for GitHub Actions workflow definitions; consider moving this environment file to the repo root or a dedicated configuration directory.

Copilot uses AI. Check for mistakes.
@KeyMatrix KeyMatrix disabled auto-merge May 13, 2025 08:19
@KeyMatrix
Copy link
Owner Author

This pull request introduces a new Conda environment configuration file to streamline dependency management for the project. The file specifies the Python version, key dependencies, and the use of both defaults and conda-forge channels.

Dependency management:

  • .github/workflows/requirements.txt: Added a Conda environment configuration file named treeom-env, which includes Python 3.10, Flask 2.1.3, WebSocket client 1.5.1, Pytest 7.4.2, Flake8 6.1.0, and support for additional dependencies via pip and requirements.txt.

@KeyMatrix KeyMatrix requested a review from Copilot May 14, 2025 00:31
Copy link

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

This PR adds a new configuration file in the GitHub workflows directory that defines a conda environment for use in the repository. Key changes include:

  • Adding environment name and channels.
  • Defining Python and package dependencies.
  • Including pip-based dependency installation.
Comments suppressed due to low confidence (1)

.github/workflows/requirements.txt:1

  • [nitpick] The file extension '.txt' might be misleading if this file is intended to be used as a conda environment configuration. Consider renaming it to 'environment.yml' to align with standard naming conventions.
name: treeom-env

@KeyMatrix KeyMatrix requested a review from Copilot May 15, 2025 01:08
Copy link

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

This PR adds a new configuration file to define the conda environment used in the GitHub workflows.

  • Added a conda environment configuration with specified channels and dependencies
  • Specifies pip dependencies via an inline requirements file
Comments suppressed due to low confidence (1)

.github/workflows/requirements.txt:1

  • [nitpick] The file name 'requirements.txt' suggests a plain text file for pip requirements, but the content defines a conda environment in YAML format. Consider renaming the file to 'environment.yml' for clearer intent.
name: treeom-env

This was linked to issues May 24, 2025
@KeyMatrix KeyMatrix enabled auto-merge September 28, 2025 13:25
@KeyMatrix KeyMatrix added this to the Test milestone Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Babat bug Something isn't working documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed invalid This doesn't seem right question Further information is requested wontfix This will not be worked on

Projects

2 participants