Skip to content

[uikit-workshop] Remove copy-dist flag and logic from UIKit-Workshop #919

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

Merged
merged 2 commits into from
Aug 17, 2018
Merged

Conversation

joshbatley
Copy link
Contributor

Closes #913

Summary of changes:

  • Updated uikit-workshop's CONTRIBUTING.md to mention npm scripts rather then gulp
  • Removed copyPublic function and calls from uikit-workshop's gulpfile

@bmuenzenmeyer
Copy link
Member

hey @jbatley just checking in - I have been travelling but plan to look at this soon!

@bmuenzenmeyer
Copy link
Member

FYI this PR is still important to me but the gulpfile is actually going away completely as part of #920 - which I didn't fully anticipate both changes in such quick succession. apologies!

@joshbatley
Copy link
Contributor Author

Ahh right, makes sense not to update the gulp file if there is a plan to remove it!

@bmuenzenmeyer bmuenzenmeyer merged commit 0dfb218 into pattern-lab:dev Aug 17, 2018
@bmuenzenmeyer
Copy link
Member

Resolved! 💃 Merged!

Thanks @jbatley !

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