Skip to content

Use NLR and update depends#542

Merged
nllong merged 1 commit into
develop-v2from
update-depends-and-nlr
Apr 17, 2026
Merged

Use NLR and update depends#542
nllong merged 1 commit into
develop-v2from
update-depends-and-nlr

Conversation

@nllong

@nllong nllong commented Apr 15, 2026

Copy link
Copy Markdown
Member

Any background context you want to provide?

NREL is now NLR
Luigi has been gone for a long time

What does this PR do?

  • run poetry update
  • rename some items

How should this be manually tested?

CI should suffice

What are the relevant tickets?

Screenshots (if appropriate)

@nllong nllong added the dependencies Pull requests that update a dependency file label Apr 15, 2026
@nllong nllong requested review from Copilot and kflemin April 15, 2026 21:30
@nllong nllong added Non-breaking Change Schema: No Changes No updates to the BuildingSync Schema labels Apr 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates project metadata and documentation/examples to reflect the NREL → NLR rename, and bumps pre-commit hook revisions.

Changes:

  • Update Poetry metadata (author email domain) to use nlr.gov.
  • Rename “NREL” → “NLR” in example XML files and contributing documentation (including contact updates).
  • Bump .pre-commit-config.yaml hook revisions (pre-commit-hooks, ruff-pre-commit).

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pyproject.toml Updates author metadata to the new email domain.
migration_scripts/upgrade_from_2.4_to_2.5.py Removes noqa suppression on XML parsing call.
examples/LL87.xml Renames sample premises from NREL to NLR.
examples/Golden Test File.xml Renames sample premises from NREL to NLR.
.pre-commit-config.yaml Updates hook revisions (pre-commit hooks + ruff).
.github/CONTRIBUTING.md Updates contact info and replaces NREL references with NLR.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread migration_scripts/upgrade_from_2.4_to_2.5.py
@nllong nllong merged commit 8f86a9f into develop-v2 Apr 17, 2026
9 of 12 checks passed
@nllong nllong deleted the update-depends-and-nlr branch April 17, 2026 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file Non-breaking Change Schema: No Changes No updates to the BuildingSync Schema

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants