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 CI for Automated Releases #16

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

srhinos
Copy link

@srhinos srhinos commented Aug 17, 2024

I noticed a renewed interest in maintaining this via #13

Figured I'd open a PR as I've had a working deployment pipeline for a year-ish or so now just so I can stay up to date with things.

I'd suggest reverting or modifying the README changes I made along with the local.lua changes as I assumed the project was abandoned and I'd pick up maintenance of it!

@MChambers1992
Copy link
Owner

Hey thanks for the PR, really appreciate it. Can you revert the files as part of the PR that you suggested? Also in terms of setting the GitHub Action up does your fork have an example of what that setup looks like? I've seen this tutorial that I planned to follow for it https://wowpedia.fandom.com/wiki/Using_the_BigWigs_Packager_with_GitHub_Actions

@srhinos
Copy link
Author

srhinos commented Aug 24, 2024

Hey thanks for the PR, really appreciate it. Can you revert the files as part of the PR that you suggested? Also in terms of setting the GitHub Action up does your fork have an example of what that setup looks like? I've seen this tutorial that I planned to follow for it https://wowpedia.fandom.com/wiki/Using_the_BigWigs_Packager_with_GitHub_Actions

my fork has the setup already added with a release shown as apart of it. I havent ever really set this up to target Curseforge but for targeting Github, this works already as is!

I'll update the fork later in the day to clean up this PR to merge

@MChambers1992
Copy link
Owner

@srhinos if you update the PR based on the changes in your fork I'd be happy to merge them

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