pepperplus/pepperplus-ri
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Add environment variables #22

Merged
0950726 merged 2 commits from feat/environment-variables into dev 2025-12-13 13:43:53 +00:00
Conversation 14 Commits 2 Files Changed 10 +200 -49
0950726 commented 2025-12-10 12:35:22 +00:00 (Migrated from git.science.uu.nl)
Copy Link

Adds support for environment variables to change configuration, including docs and tests.

To verify:

  • Pipeline passes
  • Documentation is up to date
  • Tests are up to date (new code is covered)
  • Install newly added dependency (pip install -r requirements.txt)
  • Follow the instructions in the README to create and use environment variables, are the docs clear?
    • To test, run the unit tests while the AGENT__CONTROL_BACKEND_HOST is set to something other than "localhost", at least one test should fail

ref: N25B-352

Adds support for environment variables to change configuration, including docs and tests. To verify: - [x] Pipeline passes - [x] Documentation is up to date - [x] Tests are up to date (new code is covered) - [x] Install newly added dependency (`pip install -r requirements.txt`) - [x] Follow the instructions in the README to create and use environment variables, are the docs clear? - [x] To test, run the unit tests while the `AGENT__CONTROL_BACKEND_HOST` is set to something other than "localhost", at least one test should fail ref: N25B-352
0950726 commented 2025-12-10 12:35:23 +00:00 (Migrated from git.science.uu.nl)
Copy Link

assigned to @0950726

assigned to @0950726
0950726 commented 2025-12-11 12:13:26 +00:00 (Migrated from git.science.uu.nl)
Copy Link

changed the description

changed the description
s.o.h.luijkx commented 2025-12-11 14:45:23 +00:00 (Migrated from git.science.uu.nl)
Copy Link

marked the checklist item Pipeline passes as completed

marked the checklist item **Pipeline passes** as completed
s.o.h.luijkx commented 2025-12-11 14:45:25 +00:00 (Migrated from git.science.uu.nl)
Copy Link

marked the checklist item Documentation is up to date as completed

marked the checklist item **Documentation is up to date** as completed
s.o.h.luijkx commented 2025-12-11 14:45:27 +00:00 (Migrated from git.science.uu.nl)
Copy Link

marked the checklist item Tests are up to date (new code is covered) as completed

marked the checklist item **Tests are up to date (new code is covered)** as completed
s.o.h.luijkx commented 2025-12-11 14:45:28 +00:00 (Migrated from git.science.uu.nl)
Copy Link

marked the checklist item Install newly added dependency (pip install -r requirements.txt) as completed

marked the checklist item **Install newly added dependency (`pip install -r requirements.txt`)** as completed
s.o.h.luijkx commented 2025-12-11 14:45:29 +00:00 (Migrated from git.science.uu.nl)
Copy Link

marked the checklist item Follow the instructions in the README to create and use environment variables, are the docs clear? as completed

marked the checklist item **Follow the instructions in the README to create and use environment variables, are the docs clear?** as completed
s.o.h.luijkx commented 2025-12-11 14:45:31 +00:00 (Migrated from git.science.uu.nl)
Copy Link

marked the checklist item To test, run the unit tests while the AGENT__CONTROL_BACKEND_HOST is set to something other than "localhost", at least one test should fail as completed

marked the checklist item **To test, run the unit tests while the `AGENT__CONTROL_BACKEND_HOST` is set to something other than "localhost", at least one test should fail** as completed
s.o.h.luijkx commented 2025-12-11 14:45:39 +00:00 (Migrated from git.science.uu.nl)
Copy Link

approved this merge request

approved this merge request
2584433 commented 2025-12-13 13:42:40 +00:00 (Migrated from git.science.uu.nl)
Copy Link

approved this merge request

approved this merge request
2584433 commented 2025-12-13 13:43:08 +00:00 (Migrated from git.science.uu.nl)
Copy Link

added 3 commits

  • 3a259c11...017dbfaa - 2 commits from branch dev
  • a2cb2ae9 - Merge branch 'dev' of ssh://git.science.uu.nl/ics/sp/2025/n25b/pepperplus-ri...

Compare with previous version

added 3 commits <ul><li>3a259c11...017dbfaa - 2 commits from branch <code>dev</code></li><li>a2cb2ae9 - Merge branch &#39;dev&#39; of ssh://git.science.uu.nl/ics/sp/2025/n25b/pepperplus-ri...</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-ri/-/merge_requests/22/diffs?diff_id=135881&start_sha=3a259c11706864bc52d2aec4f1fa0368dd0f8888)
2584433 (Migrated from git.science.uu.nl) scheduled this pull request to auto merge when all checks succeed 2025-12-13 13:43:41 +00:00
2584433 commented 2025-12-13 13:43:53 +00:00 (Migrated from git.science.uu.nl)
Copy Link

mentioned in commit df702f1e44

mentioned in commit df702f1e4484ef2e1799cdd8338c3777edea285b
2584433 (Migrated from git.science.uu.nl) merged commit df702f1e44 into dev 2025-12-13 13:43:53 +00:00
s.o.h.luijkx (Migrated from git.science.uu.nl) approved these changes 2026-02-02 13:26:45 +00:00
2584433 (Migrated from git.science.uu.nl) approved these changes 2026-02-02 13:26:45 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
s.o.h.luijkx
2584433
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
kasper twirre pepperplus
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pepperplus/pepperplus-ri#22
Delete Branch "feat/environment-variables"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?