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

Folder restructuring #74

Merged
merged 2 commits into from Mar 12, 2018
Merged

Folder restructuring #74

merged 2 commits into from Mar 12, 2018

Conversation

fallion
Copy link
Contributor

@fallion fallion commented Mar 12, 2018

This Pull Request meets the following criteria:

  • Tests have been added/adjusted for my new feature
  • New Components are registered in index.js of my project

Closes #56

Copy link
Contributor

@jaroslav-kubicek jaroslav-kubicek left a comment

Choose a reason for hiding this comment

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

I have some ideas for improvements, but good job for now 👍

@@ -1,27 +1,48 @@
{
"name": "kiwi-ui",
"version": "0.0.0",
"name": "@kiwicom/balkan-ui",
Copy link
Contributor

Choose a reason for hiding this comment

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

why is it still kiwicom/balkan-ui?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Renaming is part of #73

In case you want to develop in your current project:

* `yarn add @kiwicom/balkan-ui`
* `yarn watch` in this folder
Copy link
Contributor

Choose a reason for hiding this comment

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

I would recommend slightly better notation here, e.g. in this folder => in the root dir of this project

@jaroslav-kubicek jaroslav-kubicek merged commit d4639ed into master Mar 12, 2018
@jaroslav-kubicek jaroslav-kubicek deleted the re-structuring-2 branch March 12, 2018 16:28
@fallion
Copy link
Contributor Author

fallion commented Mar 12, 2018

@jaroslav-kubicek I have many improvements as well, but the aim of this PR is only the restructuring with as small changes as possible. Everything else can be built on top of it.

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.

None yet

2 participants