@10up/open-source-practice would be good to get this through review if someone's free/able, thanks!

Hi @claytoncollie,

Thanks for working on this and raising the PR. Apologies for the delay in reviewing it.

I tried to dry-run this PR action on https://github.com/iamdharmesh/autoshare-for-twitter repo (fork of 10up repo) and it's failing with rsync error: syntax or usage error (code 1) (Logs can be found here: https://github.com/iamdharmesh/autoshare-for-twitter/actions/runs/10111347459/job/27963100761). Could you please help to check if you are facing the same or it is the actual issue that needs to be fixed?

Thank you.

@iamdharmesh Thanks for help testing this feature. I made one change to the --filter flag which now allows directory syntax that is used in the distignore file. The static analysis tool also caught an issue that I fixed on line 118.

@iamdharmesh back to you for review/merge... thanks @claytoncollie!

Hi @claytoncollie,

Apologies for the delay in reviewing again! Thanks for making the changes. The action is now passing without any issues for the existing .distignore files. However, I tried to test the full .gitignore syntax by following the test steps, but it doesn't seem to be working for me. Could you please help confirm if I missed anything?

I made changes to the .distignore file here: https://github.com/iamdharmesh/autoshare-for-twitter/blob/develop/.distignore to ignore all .txt files but include the readme.txt. However, readme.txt is not being included in the release zip(zip here). I'm not sure if I'm doing something wrong or missing anything. Could you please help check this?

Thanks again for all your help here.

Hi @iamdharmesh, I looked at your autoshare repo and noticed that the .distignore was ignoring ALL *.txt files, which was overriding your !readme.txt rule. When I first looked, I noticed the README.md and CONTRIBUTING.md rules worked, which made me think there was a conflict for text files. Have a look at this commit and this release to see if the issue is resolved.

iamdharmesh/autoshare-for-twitter@cbc8267

https://github.com/iamdharmesh/autoshare-for-twitter/releases/tag/0.0.9

@iamdharmesh I am re-reading your original comment about ignoring all text files and then having this single exclusion. Is that possible with .gitignore?

@iamdharmesh I am re-reading your original comment about ignoring all text files and then having this single exclusion. Is that possible with .gitignore?

Hi @claytoncollie,

Thanks for checking on this. Yes, I am excluding all text files by default and then specifically including a single file. This can be done using both .gitignore and .distignore. When we run the wp dist-archive command, it will include that one text file while ignoring all other .txt files in the created zip.

@iamdharmesh Can you test this again, please? I think this is the correct syntax.

https://github.com/iamdharmesh/autoshare-for-twitter/releases/tag/0.0.19

Hi @claytoncollie,

Thanks a lot for working on this. It looks great now and resolves the issue I reported, Great work! While comparing the before-and-after behavior, I noticed that the new sync command does not exclude or delete files that are already committed to SVN, even if they are marked for exclusion.

For example, if a plugin repository has readme.txt and the /vendor directory is already committed to WP.ORG SVN, and we exclude them using .distignore by adding *.txt or readme.txt and /vendor, they are still not being excluded. Could you please look into this one? Please let me know if any further information is needed.

Thank you!

Some links related to my test:

• .distignore file: https://github.com/iamdharmesh/autoshare-for-twitter/blob/d787cc02979463e61b598810f322ed0cc07b339e/.distignore
• Expected zip: https://github.com/iamdharmesh/autoshare-for-twitter/releases/tag/0.0.22
• Zip generated with this PR: https://github.com/iamdharmesh/autoshare-for-twitter/releases/tag/0.0.21

Hi @iamdharmesh I will have to pick this up next week.