Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updated docs functionality to account for dataclasses refactor #1716

Closed
wants to merge 7 commits into from
Closed

updated docs functionality to account for dataclasses refactor #1716

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
b52b3cf
updated docs functionality to account for dataclasses refactor
talagluck Aug 30, 2019
389cf80
fixed the ExtendDictFields dict to adhere to pep8
talagluck Aug 30, 2019
1843b64
updated test_contracts_graph_parsed.py
talagluck Sep 4, 2019
6c5874a
finished updating graph_compiled and graph_parsed tests
talagluck Sep 4, 2019
37eec31
updated more unit tests
talagluck Sep 5, 2019
4cd284b
added docs to the NodeConfig, and updated tests to passing
talagluck Sep 6, 2019
56a2e8b
Merge remote-tracking branch 'upstream/dev/louisa-may-alcott' into fe…
talagluck Nov 5, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions core/dbt/contracts/graph/parsed.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -190,6 +190,7 @@ class ParsedNodeDefaults(ParsedNodeMandatory):
columns: Dict[str, ColumnInfo] = field(default_factory=dict)
patch_path: Optional[str] = None
build_path: Optional[str] = None
docs: Dict[str, Any] = field(default_factory=dict)


@dataclass
Expand Down
2 changes: 1 addition & 1 deletion core/dbt/source_config.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

class SourceConfig:
AppendListFields = {'pre-hook', 'post-hook', 'tags'}
ExtendDictFields = {'vars', 'column_types', 'quoting', 'persist_docs'}
ExtendDictFields = {'vars', 'column_types', 'quoting', 'persist_docs', 'docs'}
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like this line failed for pep8 reasons:

core/dbt/source_config.py:10:80: E501 line too long (82 > 79 characters)

Copy link
Author

Choose a reason for hiding this comment

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

Just fixed.

Copy link
Contributor

Choose a reason for hiding this comment

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

Great, thanks! Tests are running now

ClobberFields = {
'alias',
'schema',
Expand Down