Skip to content

Update end year in license headers#1362

Merged
ktangsali merged 17 commits intoNVIDIA:mainfrom
mnabian:update_license_header
Feb 5, 2026
Merged

Update end year in license headers#1362
ktangsali merged 17 commits intoNVIDIA:mainfrom
mnabian:update_license_header

Conversation

@mnabian
Copy link
Collaborator

@mnabian mnabian commented Jan 30, 2026

PhysicsNeMo Pull Request

Description

Checklist

Dependencies

Review Process

All PRs are reviewed by the PhysicsNeMo team before merging.

Depending on which files are changed, GitHub may automatically assign a maintainer for review.

We are also testing AI-based code review tools (e.g., Greptile), which may add automated comments with a confidence score.
This score reflects the AI’s assessment of merge readiness and is not a qualitative judgment of your work, nor is
it an indication that the PR will be accepted / rejected.

AI-generated feedback should be reviewed critically for usefulness.
You are not required to respond to every AI comment, but they are intended to help both authors and reviewers.
Please react to Greptile comments with 👍 or 👎 to provide feedback on their accuracy.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 30, 2026

Too many files changed for review. (1335 files found, 100 file limit)

@mnabian
Copy link
Collaborator Author

mnabian commented Jan 30, 2026

/blossom-ci

Copy link
Collaborator

@peterdsharpe peterdsharpe left a comment

Choose a reason for hiding this comment

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

LGTM

@mnabian mnabian enabled auto-merge January 30, 2026 19:55
@megnvidia
Copy link
Collaborator

looks good to me. Approved

Copy link
Collaborator

@laserkelvin laserkelvin left a comment

Choose a reason for hiding this comment

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

CONTRIBUTING.md and benchmarks/run_benchmarks.sh still have 2023 - 2025.

Those were the two that came up when I ran a grep for "- 2025"

@mnabian
Copy link
Collaborator Author

mnabian commented Jan 31, 2026

CONTRIBUTING.md and benchmarks/run_benchmarks.sh still have 2023 - 2025.

Those were the two that came up when I ran a grep for "- 2025"

Thanks for checking! I updated those two.

@mnabian mnabian requested a review from laserkelvin January 31, 2026 00:43
@mnabian
Copy link
Collaborator Author

mnabian commented Jan 31, 2026

/blossom-ci

@mnabian
Copy link
Collaborator Author

mnabian commented Feb 3, 2026

/blossom-ci

Copy link
Collaborator

@coreyjadams coreyjadams left a comment

Choose a reason for hiding this comment

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

LGTM! @ktangsali may need to force this one through since the list of codeowners is too long.

@mnabian mnabian mentioned this pull request Feb 5, 2026
6 tasks
@mnabian mnabian disabled auto-merge February 5, 2026 00:32
@mnabian
Copy link
Collaborator Author

mnabian commented Feb 5, 2026

/blossom-ci

@coreyjadams
Copy link
Collaborator

@ktangsali or @NickGeneva I think we will need to force merge this one. Can you take a look? We have 5 approvals already.

@ktangsali ktangsali merged commit 9fa5bb5 into NVIDIA:main Feb 5, 2026
3 of 4 checks passed
@mnabian mnabian deleted the update_license_header branch February 5, 2026 01:05
@greptile-apps greptile-apps bot mentioned this pull request Feb 5, 2026
6 tasks
coreyjadams pushed a commit to coreyjadams/physicsnemo that referenced this pull request Feb 6, 2026
* update license headers- second try

* update end year in license headers

* update copyright.txt

* Update CONTRIBUTING.md

* Update run_benchmarks.sh

* resolve conflicts
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.

7 participants