Skip to content

Commit f41d379

Browse files
authored
Merge pull request #663 from dbt-msft/fix/dev-settings
fix(dev): devcontainer settings, fix keyring, add package init
2 parents d82152e + 541e65d commit f41d379

47 files changed

Lines changed: 77 additions & 20 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.devcontainer/devcontainer.json

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,23 @@
22
"name": "Python 3",
33
"image": "mcr.microsoft.com/devcontainers/python:3.10-bookworm",
44
"features": {
5-
"ghcr.io/devcontainers/features/docker-in-docker:2.17.0": {}
5+
"ghcr.io/devcontainers/features/docker-in-docker:2.17.0": {
6+
"moby": false
7+
}
68
},
7-
"forwardPorts": [1433],
9+
"forwardPorts": [
10+
1433
11+
],
812
"postStartCommand": "/bin/bash ./.devcontainer/setup_odbc.sh && /bin/bash ./.devcontainer/setup_env.sh",
13+
"customizations": {
14+
"vscode": {
15+
"settings": {
16+
"python-envs.alwaysUseUv": true,
17+
"python-envs.terminal.autoActivationType": "shellStartup",
18+
"python.defaultInterpreterPath": "${containerWorkspaceFolder}/.venv/bin/python"
19+
}
20+
}
21+
},
922
"containerEnv": {
1023
"SQLSERVER_TEST_DRIVER": "ODBC Driver 18 for SQL Server",
1124
"SQLSERVER_TEST_HOST": "127.0.0.1",

.devcontainer/setup_odbc.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
1-
curl https://packages.microsoft.com/keys/microsoft.asc | sudo tee /etc/apt/trusted.gpg.d/microsoft.asc
1+
curl https://packages.microsoft.com/keys/microsoft.asc \
2+
| gpg --dearmor \
3+
| sudo tee /usr/share/keyrings/microsoft-prod.gpg >/dev/null
24

35
#Download appropriate package for the OS version
46
#Choose only ONE of the following, corresponding to your OS version

dbt/__init__.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
from pkgutil import extend_path
2+
3+
__path__ = extend_path(__path__, __name__)

dbt/adapters/sqlserver/__init__.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
from dbt.adapters.base import AdapterPlugin
2-
32
from dbt.adapters.sqlserver.sqlserver_adapter import SQLServerAdapter
43
from dbt.adapters.sqlserver.sqlserver_column import SQLServerColumn
54
from dbt.adapters.sqlserver.sqlserver_configs import SQLServerConfigs

dbt/adapters/sqlserver/relation_configs/policies.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,9 @@
11
from dataclasses import dataclass
22

3-
from dbt.adapters.contracts.relation import Policy
43
from dbt_common.dataclass_schema import StrEnum
54

5+
from dbt.adapters.contracts.relation import Policy
6+
67
MAX_CHARACTERS_IN_IDENTIFIER = 127
78

89

dbt/adapters/sqlserver/sqlserver_adapter.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,6 @@
22

33
import agate
44
import dbt_common.exceptions
5-
from dbt.adapters.base.column import Column as BaseColumn
6-
from dbt.adapters.base.impl import ConstraintSupport
7-
from dbt.adapters.base.meta import available
8-
from dbt.adapters.base.relation import BaseRelation
9-
from dbt.adapters.capability import Capability, CapabilityDict, CapabilitySupport, Support
10-
from dbt.adapters.events.types import SchemaCreation
11-
from dbt.adapters.reference_keys import _make_ref_key_dict
12-
from dbt.adapters.sql.impl import CREATE_SCHEMA_MACRO_NAME, SQLAdapter
135
from dbt_common.behavior_flags import BehaviorFlag
146
from dbt_common.contracts.constraints import (
157
ColumnLevelConstraint,
@@ -18,6 +10,14 @@
1810
)
1911
from dbt_common.events.functions import fire_event
2012

13+
from dbt.adapters.base.column import Column as BaseColumn
14+
from dbt.adapters.base.impl import ConstraintSupport
15+
from dbt.adapters.base.meta import available
16+
from dbt.adapters.base.relation import BaseRelation
17+
from dbt.adapters.capability import Capability, CapabilityDict, CapabilitySupport, Support
18+
from dbt.adapters.events.types import SchemaCreation
19+
from dbt.adapters.reference_keys import _make_ref_key_dict
20+
from dbt.adapters.sql.impl import CREATE_SCHEMA_MACRO_NAME, SQLAdapter
2121
from dbt.adapters.sqlserver.sqlserver_column import SQLServerColumn
2222
from dbt.adapters.sqlserver.sqlserver_configs import SQLServerConfigs
2323
from dbt.adapters.sqlserver.sqlserver_connections import SQLServerConnectionManager

dbt/adapters/sqlserver/sqlserver_column.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,9 @@
11
from typing import Any, ClassVar, Dict
22

3-
from dbt.adapters.base.column import Column
43
from dbt_common.exceptions import DbtRuntimeError
54

5+
from dbt.adapters.base.column import Column
6+
67

78
class SQLServerColumn(Column):
89
TYPE_LABELS: ClassVar[Dict[str, str]] = {

dbt/adapters/sqlserver/sqlserver_connections.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -38,15 +38,15 @@ class AccessToken: # type: ignore[no-redef]
3838
ManagedIdentityCredential = None
3939
_AZURE_IDENTITY_IMPORT_ERROR = exc
4040

41-
from dbt.adapters.contracts.connection import AdapterResponse, Connection, ConnectionState
42-
from dbt.adapters.events.logging import AdapterLogger
43-
from dbt.adapters.events.types import AdapterEventDebug, ConnectionUsed, SQLQuery, SQLQueryStatus
44-
from dbt.adapters.sql.connections import SQLConnectionManager
4541
from dbt_common.clients.agate_helper import empty_table
4642
from dbt_common.events.contextvars import get_node_info
4743
from dbt_common.events.functions import fire_event
4844
from dbt_common.utils.casting import cast_to_str
4945

46+
from dbt.adapters.contracts.connection import AdapterResponse, Connection, ConnectionState
47+
from dbt.adapters.events.logging import AdapterLogger
48+
from dbt.adapters.events.types import AdapterEventDebug, ConnectionUsed, SQLQuery, SQLQueryStatus
49+
from dbt.adapters.sql.connections import SQLConnectionManager
5050
from dbt.adapters.sqlserver import __version__
5151
from dbt.adapters.sqlserver.sqlserver_credentials import SQLServerCredentials
5252

dbt/adapters/sqlserver/sqlserver_relation.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,16 @@
11
from dataclasses import dataclass, field
22
from typing import Optional, Type
33

4-
from dbt.adapters.base.relation import BaseRelation, EventTimeFilter
5-
from dbt.adapters.utils import classproperty
64
from dbt_common.exceptions import DbtRuntimeError
75

6+
from dbt.adapters.base.relation import BaseRelation, EventTimeFilter
87
from dbt.adapters.sqlserver.relation_configs import (
98
MAX_CHARACTERS_IN_IDENTIFIER,
109
SQLServerIncludePolicy,
1110
SQLServerQuotePolicy,
1211
SQLServerRelationType,
1312
)
13+
from dbt.adapters.utils import classproperty
1414

1515

1616
@dataclass(frozen=True, eq=False, repr=False)

tests/functional/adapter/dbt/test_aliases.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import pytest
2+
23
from dbt.tests.adapter.aliases import fixtures
34
from dbt.tests.adapter.aliases.test_aliases import (
45
BaseAliasErrors,

0 commit comments

Comments
 (0)