Skip to content

Comments

Add CI for Automated Releases#16

Open
srhinos wants to merge 6 commits intoMChambers1992:developfrom
curseforge-mirror:develop
Open

Add CI for Automated Releases#16
srhinos wants to merge 6 commits intoMChambers1992:developfrom
curseforge-mirror:develop

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