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

download only specific files for #245 #329

Open
ttodua opened this issue Oct 22, 2023 · 3 comments
Open

download only specific files for #245 #329

ttodua opened this issue Oct 22, 2023 · 3 comments

Comments

@ttodua
Copy link

ttodua commented Oct 22, 2023

(On top of #245 PR)

  • at this moment, the added feature compares md5 of the files to determine which were changed. However, the plugin had an additional option (checkbox or so), so if that's checked, plugin will only download the specific files (which are presented as "changed/added/removed" from github push action, from example payload it's visible which files are changed on the last commit: https://pastebin.com/raw/u8u0B4PB ) . so, at first, the plugin downloaded only those files (instead of whole repo) and then it continue doing what it does (comparing md5 etc...)

like #328 , would this speed up things for users or not?

@ttodua
Copy link
Author

ttodua commented Oct 22, 2023

@parvit

@parvit
Copy link
Contributor

parvit commented Oct 24, 2023

@ttodua If i understand correctly, the plugin should be able to understand if its being triggered by a git push and only take the files in the changeset?

@ttodua
Copy link
Author

ttodua commented Oct 24, 2023

@ttodua If i understand correctly, the plugin should be able to understand if its being triggered by a git push and only take the files in the changeset?

yes I meant that. if that wont be complex and can be done within my proposal.
if it's not possible to detect how it's being triggered by, then just maybe on trigger (would not matter what it trigger) check the last push commit and use that info

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

No branches or pull requests

2 participants