Skip to content

Fix program_enrollment_with_user_report: address field coalesce, grain tests, full column docs#1926

Merged
quazi-h merged 3 commits intoprogram-enrollment-with-user-reportfrom
copilot/sub-pr-1914
Feb 27, 2026
Merged

Fix program_enrollment_with_user_report: address field coalesce, grain tests, full column docs#1926
quazi-h merged 3 commits intoprogram-enrollment-with-user-reportfrom
copilot/sub-pr-1914

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 20, 2026

  • Fix address fields to use coalesce with combined_users2 fallback
  • Add grain validation tests (dbt_expectations.expect_compound_columns_to_be_unique) matching upstream mart's grain
  • Add comprehensive column documentation for all inherited fields (31 columns total)
  • Update YAML descriptions for address fields to reflect coalesce pattern
  • Fix pre-commit.ci yamlfmt failure: rewrap program_complete_days description to match yamlfmt output

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…d column docs

Co-authored-by: quazi-h <59845076+quazi-h@users.noreply.github.com>
Copilot AI changed the title [WIP] Add program enrollment with user report Fix program_enrollment_with_user_report: address field coalesce, grain tests, full column docs Feb 20, 2026
Copilot AI requested a review from quazi-h February 20, 2026 20:41
@quazi-h
Copy link
Copy Markdown
Contributor

quazi-h commented Feb 23, 2026

@copilot please review the failed pre-commit.ci check and address the issue

…ter output

Co-authored-by: quazi-h <59845076+quazi-h@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Feb 23, 2026

@copilot please review the failed pre-commit.ci check and address the issue

Fixed in 65b5b03. The yamlfmt hook was reformatting the line wrapping of the program_complete_days description — it moved the break point from after "to the" to after "the program". The file now matches the formatter's expected output.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • results.pre-commit.ci
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@quazi-h quazi-h marked this pull request as ready for review February 27, 2026 16:43
@quazi-h quazi-h merged commit 5271ad6 into program-enrollment-with-user-report Feb 27, 2026
2 checks passed
@quazi-h quazi-h deleted the copilot/sub-pr-1914 branch February 27, 2026 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants