Skip to content

Conversation

@mrnicegyu11
Copy link
Member

@mrnicegyu11 mrnicegyu11 commented Dec 13, 2024

What do these changes do?

  • Refactor jinja2cli calling convention: Always pass .env file path explictly (necessary preparation for grafana-terraform)
  • Remove unused grafana-image-renderer files
  • enhance scripts/Makefiles for osparc.local

Related issue/s

Related PR/s

Checklist

  • I tested and it works

@mrnicegyu11 mrnicegyu11 self-assigned this Dec 13, 2024
@mrnicegyu11 mrnicegyu11 added t:enhancement New feature or request FAST labels Dec 18, 2024
@mrnicegyu11 mrnicegyu11 added this to the Event Horizon milestone Dec 18, 2024
@mrnicegyu11 mrnicegyu11 marked this pull request as ready for review December 18, 2024 08:39
Copy link
Collaborator

@YuryHrytsuk YuryHrytsuk left a comment

Choose a reason for hiding this comment

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

Let's not update jinja definition in this PR

$(REPO_BASE_DIR)/.venv/bin/j2 --format=env $(1) $(2) -o $(3)

Otherwise, fine with me. Thanks 🙏

@mrnicegyu11 mrnicegyu11 merged commit 8c0bae8 into ITISFoundation:main Jan 3, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FAST t:enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants