Skip to content

Added python script to automatically propagate the MOLE software license in root directory to all MATLAB/Octave scripts and C++ files#342

Open
deepanshu-zade wants to merge 3 commits intomainfrom
feature/license-header-manager
Open

Added python script to automatically propagate the MOLE software license in root directory to all MATLAB/Octave scripts and C++ files#342
deepanshu-zade wants to merge 3 commits intomainfrom
feature/license-header-manager

Conversation

@deepanshu-zade
Copy link
Copy Markdown
Collaborator

What type of PR is this? (check all applicable)

  • Refactor
  • [ x ] Feature
  • Bug Fix
  • Optimization
  • Example
  • [ x ] Documentation

Description

Related Issues & Documents

QA Instructions, Screenshots, Recordings

Please replace this line with instructions on how to test your changes, a note
on the devices and browsers this has been tested on, as well as any relevant
images for UI changes.

Added/updated tests?

_We encourage you to test all code included with MOLE, including examples.

  • [ x ] Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

Read Contributing Guide and Code of Conduct

[optional] Are there any post deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

@deepanshu-zade deepanshu-zade added Documentation Improvements or additions to documentation Enhancement New feature or request C++ MATLAB/Octave labels Apr 22, 2026
Copy link
Copy Markdown
Collaborator

@Tony-Drummond Tony-Drummond left a comment

Choose a reason for hiding this comment

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

I review these changes with @deepanshu-zade and verified that it is working properly. This was a good suggestion by @pritkc and the implementation changes MATLAB/Octave script and C++ files.
We would like to first merge the branch PR #341 to main, and then this branch.

@pritkc
Copy link
Copy Markdown
Collaborator

pritkc commented Apr 27, 2026

Thank you @deepanshu-zade for the feature.

I exercised doc/utils/common/update_license.py locally: py_compile; correct insert/update/skip behavior for .m, .cpp, .h, and .hpp; recursive discovery; idempotent second pass; empty tree message; invalid CLI args; missing license path and missing target directory. All of that behaved as intended.

Approving based on these checks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C++ Documentation Improvements or additions to documentation Enhancement New feature or request MATLAB/Octave

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automatic propagation of MOLE software license to MATLAB/Octave scripts and C++

3 participants