simplify release package #1337
simplify release package #1337
Conversation
| @@ -198,9 +198,6 @@ jobs: | |||
| automatic_release_tag: "latest" | |||
| title: "${{ steps.version.outputs.version }}" | |||
| files: | | |||
| OPNPS2LD-${{ env.OPL_VERSION }}.7z | |||
There was a problem hiding this comment.
Id rather keep that for pre-release too. I remember that this was done specially for your scripts that were not able to correctly parse OPNPS2LD-${{ env.OPL_VERSION }}.7z naming
There was a problem hiding this comment.
Yes, it was intended for direct download links
But duplicating the release assets only confuses people.
And the version number is clearly visible on the heading of the release card
There was a problem hiding this comment.
But after downloading you will loose that info. If you will have tons of OPNPS2LD.7z archives its a mess to understand which one is correct. I agree that packages should be reduced, but I am against the idea to keep commit-less naming. The only reason to keep short version of archive names, is your scripts, so I advice to change scripts to be able to parse more complex naming.
lots of people not used to download from github ask wich file to download lots of times. We upload the sime file twice in all cases. as for tagged releases. only leave explicitly tagged files since these tags are not generated again
lots of people not used to download from github ask wich file to download lots of times.
We upload the sime file twice in all cases. reduce that. only leave the main package duplicated. so the release version is clearly visible
Pull Request checklist
Note: these are not necessarily requirements
Pull Request description