Skip to content
This repository was archived by the owner on Apr 15, 2024. It is now read-only.

Conversation

@mchutani-CS
Copy link
Contributor

@mchutani-CS mchutani-CS commented Aug 29, 2023

Changes to be noted in the PR:

  • cortex-python -> sensa-python
  • There were a few requirements.txt files which was pointing to 6.1.0 version of cortex-python. This PR assumes they were just not updated. So I've updated it to future 6.5.0. (Hopefully examples tests will catch any issues before merging)

@mchutani-CS mchutani-CS marked this pull request as draft August 30, 2023 05:14
@@ -1,9 +1,9 @@
"""
Copyright (c) 2021. Cognitive Scale Inc. All rights reserved.
"""
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to update the Copyright to be from TT?

import boto3
import pymongo

# cortex
Copy link
Contributor

Choose a reason for hiding this comment

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

sensa

@@ -1,3 +1,3 @@
cortex-python==6.3.1a1
sensa-python==6.3.1a1
Copy link
Contributor

Choose a reason for hiding this comment

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

Should probably update the version to 6.5.0

@@ -1,3 +1,3 @@
cortex-python==6.3.1a1
sensa-python==6.3.1a1
Copy link
Contributor

Choose a reason for hiding this comment

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

RE - 6.5.0

@@ -1,3 +1,3 @@
cortex-python==6.3.1a1
sensa-python==6.3.1a1
Copy link
Contributor

Choose a reason for hiding this comment

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

RE - 6.5.0

@@ -1,3 +1,3 @@
cortex-python
sensa-python
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should also pin the version to 6.5.0 ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants