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
[docs] Move links to the master branch #15830
Conversation
47ed458
to
66b334e
Compare
Details of bundle changes.Comparing: ef18a5d...a966a24
|
0af469e
to
a966a24
Compare
"/Users/oliviertassinari/preact-next/node_modules/jest-watch-typeahead/filename.js", | ||
"/Users/oliviertassinari/preact-next/node_modules/jest-watch-typeahead/testname.js" | ||
"/Users/oliviertassinari/preact/node_modules/jest-watch-typeahead/filename.js", | ||
"/Users/oliviertassinari/preact/node_modules/jest-watch-typeahead/testname.js" |
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.
?
|
||
We will do our best to keep `next` in good shape, with tests passing at all times. | ||
We will do our best to keep `master` in good shape, with tests passing at all times. | ||
However, in order to move fast, we will make API changes that your application might not be compatible with. |
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.
Is that true for master
?
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.
Oh no, we don't 🙏!
### How to increase the chance of being accepted? | ||
|
||
We will only accept a pull request for which all tests pass. Make sure the following is true: | ||
- The branch is targeted at: | ||
- `master` for important fixes. | ||
- `next` for everything else including breaking changes. | ||
- `master` for ongoing development. | ||
- The branch is not behind its target. | ||
- If a breaking change is introduced: |
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.
?
Merging and deploying. I'm running a 3rd SEO crawl. |
It's a shame github doesn't notify you when attempting to merge a PR with an in-progress review. |
😢. At least, it's not lost, could you open a pull request? I suspect we will uncover more problems in the coming days. |
What should this be instead? "watchPlugins": [
"/Users/oliviertassinari/preact/node_modules/jest-watch-typeahead/filename.js",
"/Users/oliviertassinari/preact/node_modules/jest-watch-typeahead/testname.js"
] |
I would say, that we can remove this JSON key. Well spotted! |
next
branch, we need to change the links to point to the master branch