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

Fix magefile #1

Merged
merged 1 commit into from
Jun 23, 2022
Merged

Fix magefile #1

merged 1 commit into from
Jun 23, 2022

Conversation

carolynvs
Copy link
Contributor

I've updated the magefile with a couple changes that got the mixin working:

  • Use get.porter.sh/magefiles instead of get.porter.sh/porter/mage. The logic from the old package was split into a new repositroy and improved.
  • Use the most recent version of Porter.
  • Remove customizations to the build target.

Feel free to either merge or just grab the relevant changes that you want to keep. Sorry about the trouble, I'll get the skeletor repository fixed so that it's using the changes above. I'm sorry that it was out of date and wasted a bunch of time trying to get it to work... 😞

I've updated the magefile with a couple changes that got the mixin
working:

* Use get.porter.sh/magefiles instead of get.porter.sh/porter/mage. The
  logic from the old package was split into a new repositroy and
  improved.
* Use the most recent version of Porter.
* Remove customizations to the build target.

Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>
@richbai90 richbai90 merged commit 9843d7c into richbai90:main Jun 23, 2022
@carolynvs carolynvs deleted the fix-magefile branch June 23, 2022 21:04
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