Skip to content

chore: no more re-exporting makes import pudl go faster#5079

Closed
jdangerx wants to merge 3 commits intomainfrom
no-more-re-exporting
Closed

chore: no more re-exporting makes import pudl go faster#5079
jdangerx wants to merge 3 commits intomainfrom
no-more-re-exporting

Conversation

@jdangerx
Copy link
Copy Markdown
Member

Overview

Closes #XXXX.

What problem does this address?

What did you change?

Documentation

Make sure to update relevant aspects of the documentation:

  • Update the release notes: reference the PR and related issues.
  • Update relevant Data Source jinja templates (see docs/data_sources/templates).
  • Update relevant table or source description metadata (see src/metadata).
  • Review and update any other aspects of the documentation that might be affected by this PR.

Testing

How did you make sure this worked? How can a reviewer verify this?

To-do list

  • If updating analyses or data processing functions: make sure to update row count expectations in dbt tests.
  • Run pixi run pre-commit-run to run linters and static code analysis checks.
  • Run pixi run pytest-ci locally to ensure that the merge queue will accept your PR.
  • Review the PR yourself and call out any questions or issues you have.
  • For PRs that change the PUDL outputs significantly, run the full ETL locally and then run the data validations using dbt. If you can't run the ETL locally then run the build-deploy-pudl GitHub Action manually and ensure that it succeeds.

@jdangerx jdangerx moved this from New to In progress in Catalyst Megaproject Mar 11, 2026
@jdangerx jdangerx moved this from In progress to Backlog in Catalyst Megaproject Mar 11, 2026
@jdangerx
Copy link
Copy Markdown
Member Author

jdangerx commented Apr 1, 2026

We'll just re-do this when the time comes - #5124 probably is a good thing to get in first.

@jdangerx jdangerx closed this Apr 1, 2026
@github-project-automation github-project-automation bot moved this from Backlog to Done in Catalyst Megaproject Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant