Skip to content

feat: qdrant grist migration#227

Merged
leoguillaume merged 4 commits intomainfrom
grist-migration-qdrant
Apr 16, 2025
Merged

feat: qdrant grist migration#227
leoguillaume merged 4 commits intomainfrom
grist-migration-qdrant

Conversation

@leoguillaume
Copy link
Copy Markdown
Member

@leoguillaume leoguillaume commented Apr 15, 2025

No description provided.

github-advanced-security[bot]
github-advanced-security bot found potential problems Apr 15, 2025
View reviewed changes
app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py


# revision identifiers, used by Alembic.
revision: str = "a4ac45e7c990"

Check notice

Code scanning / CodeQL

Unused global variable Note

The global variable 'revision' is not used.

Copilot Autofix

AI 12 months ago

Copilot could not generate an autofix suggestion

Copilot could not generate an autofix suggestion for this alert. Try pushing a new commit or if the problem persists contact support.

app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py

# revision identifiers, used by Alembic.
revision: str = "a4ac45e7c990"
down_revision: Union[str, None] = "8c5ae2a3d4d0"

Check notice

Code scanning / CodeQL

Unused global variable Note

The global variable 'down_revision' is not used.

Copilot Autofix

AI 12 months ago

Copilot could not generate an autofix suggestion

Copilot could not generate an autofix suggestion for this alert. Try pushing a new commit or if the problem persists contact support.

app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py
# revision identifiers, used by Alembic.
revision: str = "a4ac45e7c990"
down_revision: Union[str, None] = "8c5ae2a3d4d0"
branch_labels: Union[str, Sequence[str], None] = None

Check notice

Code scanning / CodeQL

Unused global variable Note

The global variable 'branch_labels' is not used.

Copilot Autofix

AI 12 months ago

The best way to fix the problem is to remove the unused variable branch_labels from the code. This will eliminate the unnecessary clutter and make the code cleaner. Since the variable is not used anywhere in the code, removing it will not affect the existing functionality.

To implement the changes:

  1. Remove the line defining branch_labels from the file app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py.
Suggested changeset 1
app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py b/app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py
--- a/app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py
+++ b/app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py
@@ -16,3 +16,3 @@
 down_revision: Union[str, None] = "8c5ae2a3d4d0"
-branch_labels: Union[str, Sequence[str], None] = None
+
 depends_on: Union[str, Sequence[str], None] = None
EOF
@@ -16,3 +16,3 @@
down_revision: Union[str, None] = "8c5ae2a3d4d0"
branch_labels: Union[str, Sequence[str], None] = None

depends_on: Union[str, Sequence[str], None] = None
Copilot is powered by AI and may make mistakes. Always verify output.
app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py
revision: str = "a4ac45e7c990"
down_revision: Union[str, None] = "8c5ae2a3d4d0"
branch_labels: Union[str, Sequence[str], None] = None
depends_on: Union[str, Sequence[str], None] = None

Check notice

Code scanning / CodeQL

Unused global variable Note

The global variable 'depends_on' is not used.

Copilot Autofix

AI 12 months ago

The best way to fix the problem is to remove the unused variable depends_on from the code. This will clean up the code and eliminate the unnecessary variable definition. Since the variable is not used anywhere in the provided code snippet, removing it will not affect the existing functionality.

Suggested changeset 1
app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py b/app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py
--- a/app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py
+++ b/app/alembic/versions/2025_04_15_1513-a4ac45e7c990_remove_unique_name_on_token_and_.py
@@ -17,3 +17,2 @@
 branch_labels: Union[str, Sequence[str], None] = None
-depends_on: Union[str, Sequence[str], None] = None
 
EOF
@@ -17,3 +17,2 @@
branch_labels: Union[str, Sequence[str], None] = None
depends_on: Union[str, Sequence[str], None] = None

Copilot is powered by AI and may make mistakes. Always verify output.
@leoguillaume leoguillaume force-pushed the grist-migration-qdrant branch 4 times, most recently from ce458ae to 7d082d0 Compare April 15, 2025 15:53
@leoguillaume leoguillaume force-pushed the grist-migration-qdrant branch from 7d082d0 to a16cde1 Compare April 15, 2025 16:02
@leoguillaume leoguillaume force-pushed the grist-migration-qdrant branch 5 times, most recently from 85a7d46 to 1754e43 Compare April 16, 2025 12:07
@leoguillaume leoguillaume force-pushed the grist-migration-qdrant branch from 0ea78e4 to 1b5c302 Compare April 16, 2025 12:21
@leoguillaume leoguillaume merged commit e935d6d into main Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@leoguillaume @github-advanced-security