Skip to content

Add NFP for Deprecation Marking#11647

Open
jasondegraw wants to merge 2 commits into
developfrom
deprecation-marking
Open

Add NFP for Deprecation Marking#11647
jasondegraw wants to merge 2 commits into
developfrom
deprecation-marking

Conversation

@jasondegraw

@jasondegraw jasondegraw commented Jun 18, 2026

Copy link
Copy Markdown
Member

Pull request overview

Description of the purpose of this PR

This PR adds an NFP for a small change to the way that the obsolete IDD tag is handled so that the deprecation process can be handled a bit more rationally and a bit more like other things are handled.

Pull Request Author

  • Title of PR should be user-synopsis style (clearly understandable in a standalone changelog context)
  • Label the PR with at least one of: Defect, Refactoring, NewFeature, Performance, and/or DoNoPublish

Reviewer

  • Perform a Code Review on GitHub
  • If branch is behind develop, merge develop and build locally to check for side effects of the merge
  • If defect, verify by running develop branch and reproducing defect, then running PR and reproducing fix
  • If feature, test running new feature, try creative ways to break it
  • CI status: all green or justified
  • Check that performance is not impacted (CI Linux results include performance check)
  • Run Unit Test(s) locally
  • Check any new function arguments for performance impacts
  • Verify IDF naming conventions and styles, memos and notes and defaults
  • If new idf included, locally check the err file and other outputs

@jasondegraw jasondegraw added DoNotPublish Includes changes that shouldn't be reported in the changelog NewFeature Includes code to add a new feature to EnergyPlus labels Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DoNotPublish Includes changes that shouldn't be reported in the changelog NewFeature Includes code to add a new feature to EnergyPlus

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants