Skip to content

Conversation

@tmoody
Copy link
Member

@tmoody tmoody commented Nov 27, 2025

Issue

Macro mp_stripdiff relies upon the mp_ds2squeeze macro, but does not reference it in the program header causing the sasjs c compile stage to omit the dependency, resulting in runtime problems.

Intent

Insert the dependency

Implementation

Add dependency to the <h4> SAS Macros </h4> section of the program header.

Checks

  • Code is formatted correctly (sasjs lint).
  • Any new functionality has been unit tested.
  • All unit tests are passing (sasjs test).
  • The PR desc or underlying commits follow the Conventional Commit standard

@tmoody tmoody requested a review from allanbowe November 27, 2025 06:01
@tmoody tmoody self-assigned this Nov 27, 2025
Copy link
Member

@allanbowe allanbowe left a comment

Choose a reason for hiding this comment

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

great spot!

@allanbowe allanbowe merged commit bf6bede into main Nov 27, 2025
1 of 2 checks passed
@allanbowe allanbowe deleted the mp_stripdiff_fix branch November 27, 2025 13:27
@github-actions
Copy link

🎉 This PR is included in version 4.59.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants