Skip to content

Conversation

@sravan-s
Copy link
Contributor

@sravan-s sravan-s commented Aug 26, 2021

Before, we had internal repos for development and release
From now on, we will be developing here

Work done:

  • Fix config in pakage.json and rollup to support release process
  • Minor cleanup in README.md

@sravan-s sravan-s requested a review from HoonBaek August 26, 2021 07:18
@sravan-s sravan-s self-assigned this Aug 26, 2021
@sravan-s sravan-s force-pushed the feat/UK-859/merge-repos branch from 81b83d1 to 08b9cc3 Compare August 26, 2021 07:19
@sravan-s sravan-s changed the title Cleanup package.json [UK-859] Use this repo as the primary one Aug 26, 2021
@sravan-s sravan-s force-pushed the feat/UK-859/merge-repos branch from 08b9cc3 to 4b7638e Compare August 26, 2021 07:31
# to do: index.css cannot be bundled into ./dist using rollup.json
# Also, rollup-copy is not working synchronously
# Solution - for v3, we should change the bundling position/import of CSS
mkdir dist
Copy link
Contributor

Choose a reason for hiding this comment

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

why make directory 'dist' here? I think it's already made by rollup-c. no?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It doesn't :( There are better ways to solve this, lets do it for v3.

@sravan-s sravan-s force-pushed the feat/UK-859/merge-repos branch from 4b7638e to 5acd906 Compare August 27, 2021 01:32
@sravan-s sravan-s merged commit b7e03fe into main Aug 27, 2021
@sravan-s sravan-s deleted the feat/UK-859/merge-repos branch August 27, 2021 01:45
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.

3 participants