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

TUP-271: Use Core-Styles from TUP-UI #495

Merged
merged 16 commits into from
Jun 28, 2022

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Jun 1, 2022

Overview

Load core styles from new monorepo instead of recent standalone repo.

Background

For TUP, a monorepo (TACC/tup-ui) houses core styles, core components, and new UI client.

Related

Changes

Testing

See How to Test Core-Styles Version Change.

Screenshots

TUP-271-cms.log

first build count second build count
main tup-271

@wesleyboar wesleyboar marked this pull request as ready for review June 1, 2022 18:01
package-lock.json Outdated Show resolved Hide resolved
@wesleyboar wesleyboar changed the title feat(tup-271): use TACC/tup-ui:/libs/core-styles feat(tup-271): use core-styles from tup-ui Jun 1, 2022
@wesleyboar wesleyboar changed the title feat(tup-271): use core-styles from tup-ui FP-1647/TUP-271: Use Core-Styles from TUP-UI Jun 1, 2022
@wesleyboar wesleyboar requested review from taoteg and removed request for jarosenb, jchuahtacc, rstijerina and taoteg June 1, 2022 21:38
@wesleyboar wesleyboar marked this pull request as draft June 1, 2022 21:56
@wesleyboar wesleyboar marked this pull request as ready for review June 23, 2022 20:12
@wesleyboar wesleyboar changed the title FP-1647/TUP-271: Use Core-Styles from TUP-UI TUP-271: Use Core-Styles from TUP-UI Jun 23, 2022
@wesleyboar wesleyboar merged commit 3f54ca6 into main Jun 28, 2022
@wesleyboar wesleyboar deleted the task/tup-271-move-core-styles-to-monorepo branch June 28, 2022 22:56
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