Skip to content
This repository has been archived by the owner on Nov 19, 2020. It is now read-only.

Convert from Make to Mage #66

Merged
merged 9 commits into from
Apr 9, 2020
Merged

Convert from Make to Mage #66

merged 9 commits into from
Apr 9, 2020

Conversation

aknuds1
Copy link
Contributor

@aknuds1 aknuds1 commented Apr 6, 2020

Convert from Make to Mage. Use SDK's Mage logic.

Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
@aknuds1 aknuds1 requested a review from kylebrandt April 6, 2020 19:40
@ryantxu
Copy link
Member

ryantxu commented Apr 8, 2020

Dooh sorry did not see this PR... either use this or #68

the common stuff expects src/plugin.json so I moved it there

@aknuds1
Copy link
Contributor Author

aknuds1 commented Apr 8, 2020

@ryantxu I will try to consolidate the two PRs :)

Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
Magefile.go Outdated Show resolved Hide resolved
Magefile.go Show resolved Hide resolved
@aknuds1 aknuds1 changed the title WIP: Convert from Make to Mage Convert from Make to Mage Apr 9, 2020
Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
@aknuds1 aknuds1 marked this pull request as ready for review April 9, 2020 17:10
Co-Authored-By: Ryan McKinley <ryantxu@gmail.com>
Magefile.go Outdated Show resolved Hide resolved
Magefile.go Outdated Show resolved Hide resolved
Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
@aknuds1
Copy link
Contributor Author

aknuds1 commented Apr 9, 2020

Thanks @kylebrandt :)

Copy link
Contributor

@kylebrandt kylebrandt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haven't tested it, but looks good to me :-) If you need to test it just let me know.

@aknuds1
Copy link
Contributor Author

aknuds1 commented Apr 9, 2020

@kylebrandt Sure, but I'm getting feedback from @ryantxu on this one, so I think we're covered :)

Copy link
Member

@ryantxu ryantxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great -- in a next step, l think we should move the CI related stuff (makeing a zip) to the build-pipeline tool.

@ryantxu ryantxu merged commit f282bea into master Apr 9, 2020
@aknuds1
Copy link
Contributor Author

aknuds1 commented Apr 9, 2020

@ryantxu Yep!!

@aknuds1 aknuds1 deleted the chore/mage branch April 9, 2020 18:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants