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.

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

Add fix wrong agent import command #165

Merged
merged 9 commits into from
Jan 3, 2025

Conversation

OvidiuCode
Copy link
Contributor

Stable release checklist:

  • Updated the version using python -m dev set-version {version}
  • Updated /docs/changelog.md with the changes for the release

Pre-release checklist:

  • Updated the Unreleased section of /docs/changelog.md with the new changes.


# Move files and directories from the versioned folder to the root action directory
for item in temp_extract_path.rglob("*"):
if item.is_file():
Copy link
Contributor

Choose a reason for hiding this comment

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

I think that there's a metadata json which should be ignored here too (not just the one from the agent package).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't see any while building zip and then extracting it but I guess it could happen in other scenarios, will include that exclusion

Copy link
Contributor

Choose a reason for hiding this comment

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

There's no real issue, the only problem is that this metadata file will become outdated as the user changes the code (the metadata is created in the zip when the action package is created and shouldn't be shown when it's extracted later on).


# Move files and directories from the versioned folder to the root action directory
for item in temp_extract_path.rglob("*"):
if item.is_file() and item.name != "metadata.json":
Copy link
Contributor

Choose a reason for hiding this comment

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

This isn't the proper name of the json. I don't remember now the exact name, but you should be able to check it when you build the action package in the test (please also fix the test to check for this exclusion).

@OvidiuCode OvidiuCode merged commit e336697 into master Jan 3, 2025
17 checks passed
@OvidiuCode OvidiuCode deleted the feature/fix-wrong-agent-import branch January 3, 2025 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants