-
Notifications
You must be signed in to change notification settings - Fork 5
[BREAKING CHANGES] New base setup #391
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
base: main
Are you sure you want to change the base?
Conversation
modified: package-lock.json modified: package.json modified: src/main.ts
0c9b29a
to
e3d89f0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks very promising to me. Nice Work!
I added a few questions, otherwise I feel like we should merge this as soon as possible.
.github/workflows/test.yml
Outdated
|
||
on: | ||
push: | ||
branches: ['feature/new-base-setup'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: Can we update this so it works on all future branches?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, we need to check if we can use wildcards
id: 'theme-02', | ||
selected: false, | ||
}, | ||
], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question This is just as an example right? Because we now only have one default theme in our template.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah yes, we could remove it I guess
Pull request
This merge request extracts the styleguide and the base scss setup in new reusable repositories. The vue-template will be a boilerplate how to setup a project with all dev-tools outsourced and versioned in the future.
Ticket
ticket-link
Browser testing
Checklist
Review/Test checklist