Skip to content

Enhance persist_docs.sql#299

Merged
99Lys merged 8 commits into
dremio:mainfrom
rfeith2:enhance_persist_docs
Jul 24, 2025
Merged

Enhance persist_docs.sql#299
99Lys merged 8 commits into
dremio:mainfrom
rfeith2:enhance_persist_docs

Conversation

@rfeith2
Copy link
Copy Markdown
Contributor

@rfeith2 rfeith2 commented Jul 15, 2025

Summary

Enhances the persist docs functionality to wrap all yml info in a markdown format and then sends that through to the Dremio wiki. This enables additional metadata to be captured such as field level info.

Description

Test Results

Tested locally on my dbt project and haven't run into any issues.

Changelog

  • Added a summary of what this PR accomplishes to CHANGELOG.md

Contributor License Agreement

Related Issue

Enhances the persist docs functionality to wrap all yml info in a markdown format and then sends that through to the Dremio wiki. This enables additional metadata to be captured such as field level info.
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jul 15, 2025

CLA assistant check
All committers have signed the CLA.

@rfeith2 rfeith2 marked this pull request as ready for review July 15, 2025 19:53
@howareyouman howareyouman requested a review from 99Lys July 16, 2025 15:21
Copy link
Copy Markdown
Contributor

@99Lys 99Lys left a comment

Choose a reason for hiding this comment

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

Hi @rfeith2
Thank you for your contribution!
This enhancement looks good to me, however the respective tests need to be updated as well - as they were expecting only the model description.
Also please include a summary of your changes in the changelog.

rfeith2 added 3 commits July 17, 2025 20:41
add v1.9.1. version and persist doc enhancement
…sist_docs

Update persist docs tests for new wiki format
@rfeith2 rfeith2 requested a review from 99Lys July 21, 2025 20:37
Comment thread tests/functional/adapter/dremio_specific/test_persist_docs.py
Comment thread tests/functional/adapter/dremio_specific/test_persist_docs.py Outdated
updating version to 2 in _assert_view_wikis_and_tags_delete
Comment thread tests/functional/adapter/dremio_specific/test_persist_docs.py
@99Lys 99Lys added the enhancement New feature or request label Jul 24, 2025
99Lys
99Lys previously approved these changes Jul 24, 2025
Copy link
Copy Markdown
Contributor

@99Lys 99Lys left a comment

Choose a reason for hiding this comment

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

Thanks a lot for this !

Comment thread tests/functional/adapter/dremio_specific/test_persist_docs.py Outdated
@99Lys 99Lys merged commit 2258844 into dremio:main Jul 24, 2025
19 checks passed
@rfeith2 rfeith2 deleted the enhance_persist_docs branch July 24, 2025 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Development

Successfully merging this pull request may close these issues.

4 participants