Skip to content

Academy home helper for shared data directory#423

Merged
GusEllerm merged 1 commit into
academy-agents:mainfrom
GusEllerm:gusellerm-academy-home
Jun 16, 2026
Merged

Academy home helper for shared data directory#423
GusEllerm merged 1 commit into
academy-agents:mainfrom
GusEllerm:gusellerm-academy-home

Conversation

@GusEllerm

Copy link
Copy Markdown
Collaborator

While brainstorming and developing out auth for the Globus exchange, a small path helper was included in #416. This change pulls that helper out as a more general method for use across different modules, including logging and token storage.

Changes

  • Breaking (backwards incompatible changes to public interfaces)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (non-breaking change or feature addition)
  • Refactor (internal code or design clean up)
  • Documentation (no changes to the code)
  • Test (changes or additions to testing)
  • Build (change to CI workflows or build processes)
  • Package (changes to package metadata or dependency versions)

Pull Request Checklist

Please confirm the PR meets the following requirements.

  • Relevant tags are added based on the types of changes.
  • Code changes pass pre-commit (e.g., ruff, mypy, etc.).
  • Tests have been added to show the fix is effective or that the new feature works.
  • New and existing unit tests pass locally with the changes.
  • Docs have been updated and reviewed if relevant.

While brainstorming and developing out auth for the Globus exchange, a small path helper was included in academy-agents#416. This change pulls that helper out as a more general method for use across different modules, including logging and token storage.
@GusEllerm GusEllerm added enhancement New features or improvements to existing functionality refactor Code or design cleanup without user-facing changes test Adds or changes tests labels Jun 15, 2026
@GusEllerm GusEllerm requested a review from AK2000 June 15, 2026 16:25

@AK2000 AK2000 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me right now. In the future we might want to rename/refactor into something more general (i.e. environment_helper.py or something like that) but we can make a new PR when we run into that need.

@GusEllerm GusEllerm merged commit 537851a into academy-agents:main Jun 16, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New features or improvements to existing functionality refactor Code or design cleanup without user-facing changes test Adds or changes tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants