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

WIP on rename-zip #139

Merged
merged 1 commit into from
Nov 7, 2024
Merged

WIP on rename-zip #139

merged 1 commit into from
Nov 7, 2024

Conversation

afragen
Copy link
Contributor

@afragen afragen commented Nov 5, 2024

Pull Request

What changed?

Updated releases.yml to use softprops/action-gh-release
Used variables to use repository name and current tag for naming.

Why did it change?

Updated from outdated action. More universal.

See #135, #140

CERTIFICATION

By opening this pull request, I do agree to abide by
the CODE OF CONDUCT and be bound by the terms
of the Contribution Guidelines in effect on the date and time
of my contribution as proven by the
revision information in GitHub. I also agree that any previous contributions shall be deemed subject to the terms of the
version in effect on the date and time of this pull request, or any future revisions for pull requests I may submit.
Further, I certify that this work is my own, is original, does not violate the intellectual property of any other person
or entity, and I am not violating any license agreements or contracts I have with any person or entity. Finally, I agree
that this code may be licensed under any license deemed appropraite by AspirePress, including but not
limited to open source, closed source, proprietary or custom licenses, and that such license terms neither violate my
rights or my copyright to this code.

@afragen afragen marked this pull request as ready for review November 5, 2024 18:48
@asirota asirota requested a review from namithj November 5, 2024 18:54
@namithj
Copy link
Contributor

namithj commented Nov 5, 2024

Do not merge until #140 is in

@afragen
Copy link
Contributor Author

afragen commented Nov 5, 2024

This PR doesn't rely on a change to the repo name. The zip will be named based upon the repo name. That's what I was pointing out.

@namithj
Copy link
Contributor

namithj commented Nov 7, 2024

Let's get this merged in @asirota

@asirota asirota merged commit d2e05ff into aspirepress:main Nov 7, 2024
3 checks passed
@afragen afragen deleted the releases-yml-update branch November 7, 2024 20:56
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.

3 participants