Skip to content

Conversation

@rabea-al
Copy link
Contributor

@rabea-al rabea-al commented Oct 8, 2024

Description

  • Updated README.md to maintain consistent formatting across all files.
  • Locked package version in requirements.txt to prevent conflicts.
  • Reloaded canvas to ensure all modifications are applied.
  • Bug fixes

Pull Request Type

  • Xircuits Component Library Code
  • Workflow Example
  • Documentation
  • Others (Please Specify) lock requirements

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Tests

  • Check Readme and run the examples

Tested on?

  • Windows
  • Linux fedora
  • Centos
  • Mac
  • Others (State here -> xxx )

Copy link
Member

@MFA-X-AI MFA-X-AI left a comment

Choose a reason for hiding this comment

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

Looks like there's a one line git conflict here. Gotta resolve that before merging.

@rabea-al rabea-al requested a review from MFA-X-AI October 10, 2024 08:53
Copy link
Member

@MFA-X-AI MFA-X-AI left a comment

Choose a reason for hiding this comment

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

The test is failing, there's a typo in the requirements.txt.
I believe you can run the test on your fork, right? Let me know if it doesn't or you need help setting it up.

@rabea-al rabea-al requested a review from MFA-X-AI October 15, 2024 11:48
@MFA-X-AI MFA-X-AI merged commit 3f949ae into XpressAI:main Oct 16, 2024
3 checks passed
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.

3 participants