Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[minor] migrate to Jinjanator #1469

Merged
merged 11 commits into from
Feb 25, 2025
Merged

[minor] migrate to Jinjanator #1469

merged 11 commits into from
Feb 25, 2025

Conversation

boonware
Copy link
Contributor

@boonware boonware commented Feb 13, 2025 •
edited
Loading

MASCORE-5642

  • Migrate to jinjanator. See [minor] install jinjanator cli-base#18 for further information.
  • A utility function jinja is defined which calls Jinjanator with standard arguments. Other use cases such as passing parameter files can call Jinjanator directly.
  • Refactors the templates for Group Sync Operator and IBM RBAC to use the new custom filter.

@boonware boonware changed the title [minor] migrate to jinjanate [minor] migrate to Jinjanator Feb 13, 2025
@boonware boonware marked this pull request as ready for review February 18, 2025 10:56
@whitfiea whitfiea merged commit d9aa806 into master Feb 25, 2025
12 checks passed
@whitfiea whitfiea deleted the 5642 branch February 25, 2025 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@whitfiea whitfiea whitfiea approved these changes

@durera durera Awaiting requested review from durera durera is a code owner

@JonahLuckett JonahLuckett Awaiting requested review from JonahLuckett JonahLuckett is a code owner

@andrercm andrercm Awaiting requested review from andrercm andrercm is a code owner

@sanjayprab sanjayprab Awaiting requested review from sanjayprab sanjayprab is a code owner

@terenceq terenceq Awaiting requested review from terenceq terenceq is a code owner

@alequint alequint Awaiting requested review from alequint alequint is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@boonware @whitfiea