This repository has been archived by the owner on Jan 26, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 13
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #3932 from DivanteLtd/develop
Release v1.11
- Loading branch information
Showing
699 changed files
with
34,278 additions
and
13,268 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,3 @@ | ||
core/build/*.js | ||
node_modules | ||
packages/module/*.js |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -33,3 +33,4 @@ core/resource/i18n/ru-RU.json | |
|
||
#unit testing | ||
/test/unit/coverage | ||
/static |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
const tasks = arr => arr.join(' && ') | ||
|
||
module.exports = { | ||
'hooks': { | ||
'pre-commit': tasks([ | ||
'lint-staged' | ||
]), | ||
'pre-push': tasks([ | ||
'yarn test:unit' | ||
]) | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
module.exports = { | ||
"*.{js,vue,ts}": "eslint", | ||
"**/i18n/*.csv": ["node ./core/scripts/utils/sort-translations.js", "git add"] | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,11 +1,11 @@ | ||
# How to Contribute | ||
|
||
Already a JS/Vue.js developer? Pick an issue, push a PR and instantly become a member of the vue-storefront contributors community. | ||
Already a JavaScript/Vue.js developer? Pick an issue, push a pull request (PR) and instantly become a member of the vue-storefront contributors community. | ||
We've marked some issues as "Easy first pick" to make it easier for newcomers to begin! | ||
|
||
Thank you for your interest in, and engagement! | ||
|
||
Before you type an issue please read about out [release lifecycle](https://docs.vuestorefront.io/guide/basics/release-cycle.html) | ||
Before you type an issue please read about out [release lifecycle](https://docs.vuestorefront.io/guide/basics/release-cycle.html). | ||
|
||
# Branches | ||
|
||
|
@@ -18,41 +18,39 @@ The main branches used by the core team are: | |
|
||
Please use "develop" or "RC" for development purposes as the "master" can be merged just as the new release is coming out (about once a month)! | ||
|
||
## Issue reporting guidelines: | ||
## Issue Reporting Guidelines | ||
|
||
Always define the type of issue: | ||
* Bug report | ||
* Feature request | ||
|
||
While writing issues, be as specific as possible | ||
All requests regarding support with implementation or application setup should be sent to [email protected] | ||
While writing issues, be as specific as possible. All requests regarding support with implementation or application setup should be sent to [email protected]. | ||
|
||
**Tag your issues properly**. If you found a bug tag it with `bug` label. If you're requesting new feature tag it with `feature request` label. | ||
**Tag your issues properly**. If you found a bug, tag it with `bug` label. If you're requesting new feature, tag it with `feature request` label. | ||
|
||
## Git flow | ||
We're introducing TypeScript to Vue Storefront core, so You can use it where it's appropriate - but please be pragmatic. | ||
Here are some thoughts on how to use TS features in Vue Storefront: [TypeScript Action Plan](https://github.com/DivanteLtd/vue-storefront/blob/master/doc/TypeScript%20Action%20Plan.md). | ||
## Git Flow | ||
|
||
## Pull request Checklist | ||
We're introducing TypeScript to Vue Storefront core, so you can use it where it's appropriate - but please be pragmatic. | ||
Here are some thoughts on how to use TypeScript features in Vue Storefront: [TypeScript Action Plan](https://github.com/DivanteLtd/vue-storefront/blob/master/doc/TypeScript%20Action%20Plan.md). | ||
|
||
Here’s how to submit a pull request. <b>Pull request that don't meet these requirements will not be merged.</b> | ||
## Pull Request Checklist | ||
|
||
**ALWAYS** use [Pull Request template](https://github.com/DivanteLtd/vue-storefront/blob/master/PULL_REQUEST_TEMPLATE.md) it's automatically added to each PR. | ||
1. Fork the repository and clone it locally fro the 'develop' branch. Make sure it's up to date with current `develop` branch | ||
1. Fork the repository and clone it locally from the 'develop' branch. Make sure it's up to date with current `develop` branch | ||
2. Create a branch for your edits. Use the following branch naming conventions: | ||
* bugfix/task-title | ||
* feature/task-name | ||
3. Use Pull Request template and fill as much fields as possible to describe your solution. | ||
4. Reference any relevant issues or supporting documentation in your PR (ex. “Issue: 39. Issue title.”). | ||
5. If you are adding new feature provide documentation along with the PR. Also, add it to [upgrade notes](https://github.com/DivanteLtd/vue-storefront/blob/master/doc/Upgrade%20notes.md) | ||
6. If you are removing/renaming something or changing its behavior also include it in [upgrade notes](https://github.com/DivanteLtd/vue-storefront/blob/master/doc/Upgrade%20notes.md) | ||
7. Test your changes! Run your changes against any existing tests and create new ones when needed. Make sure your changes don’t break the existing project. Make sure that your branch is passing Travis CI build. | ||
7. Test your changes! Run your changes against any existing tests and create new ones when needed. Make sure your changes don’t break the existing project. Make sure that your branch is passing Travis CI build. | ||
8. If you have found a potential security vulnerability, please DO NOT report it on the public issue tracker. Instead, send it to us at [email protected]. We will work with you to verify and fix it as soon as possible. | ||
(https://github.com/DivanteLtd/vue-storefront/blob/master/README.md#documentation--table-of-contents)) | ||
|
||
## Acceptance criteria | ||
## Acceptance Criteria | ||
|
||
Your pull request will be merged after meeting following criteria: | ||
- Everything from "Pull Request Checklist" | ||
- PR is proposed to appropriate branch | ||
- PR is proposed to appropriate branch | ||
- There are at least two approvals from core team members |
Oops, something went wrong.