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

Update error codes and results.json #10619

Merged
merged 2 commits into from
Sep 6, 2017

Conversation

flarnie
Copy link
Contributor

@flarnie flarnie commented Sep 6, 2017

Preparing to do an RC, just updating these files.

@flarnie flarnie added this to the 16.0 milestone Sep 6, 2017
@flarnie flarnie changed the title Update Yarn lockfile and results.json Update Yarn lockfile, error codes, and results.json Sep 6, 2017
@bvaughn
Copy link
Contributor

bvaughn commented Sep 6, 2017

Is this what we want to do right before an RC? (Update the Yarn lockfile, that is?)

Should we maybe only update our deps (not dev deps):

    "fbjs": "^0.8.9",
    "loose-envify": "^1.1.0",
    "object-assign": "^4.1.0",
    "prop-types": "^15.5.6"

flarnie added a commit to flarnie/react that referenced this pull request Sep 6, 2017
**what is the change?:**
This just updated the error codes on the docs.

For my future self, recording the steps it took to get here:
- On master, run `npm run build -- --extract-errors`
- Create a commit with those changes, and open a PR
  (facebook#10619)
- Cherry-pick that commit onto 15-stable, and open a PR
  (facebook#10621)
- Update the error codes on the `gh-pages branch` by doing the
  following based on docs
  (https://github.com/facebook/react/tree/master/docs#updating-facebookgithubioreact):
  - Set up a sister directory to `react` with `gh-pages` checked out,
    and call it `react-gh-pages`
  - In your `react` repo, with `15-stable` checked out and the
    error-codes update cherry-picked, run:
    `cd docs && bundle rake exec && bundle exec rake fetch_remotes && bundle exec rake release`
  - `cd ../react-gh-pages` and check in those changes, create a
    commit/PR
  - Manually test running the docs

**why make this change?:**
Even though this is just an RC, and not a final major release, we still
want the docs to be as up-to-date as possible.

**test plan:**
Run the docs locally and make sure things work.
(Flarnie will insert screenshots.)

**issue:**
facebook#8854
**what is the change?:**
Ran the build after updating Yarn lockfile and recorded latest build
size results.

**why make this change?:**
Let's get an idea of bundle size at this point, when we are preparing
the 16.0 RC.

**test plan:**
NA

**issue:**
facebook#8854
@flarnie
Copy link
Contributor Author

flarnie commented Sep 6, 2017

I'm not sure about how we want the Yarn lockfile updated - it's in the docs for doing releases, but we should decide what makes sense.

I'll remove that commit from this PR so we can get the error code and bundle size changes in faster.

@sophiebits sophiebits changed the title Update Yarn lockfile, error codes, and results.json Update error codes and results.json Sep 6, 2017
@gaearon
Copy link
Collaborator

gaearon commented Sep 6, 2017

@flarnie

Which docs are you looking at? I included different instructions about Yarn in the new guide: #10620. Please don't use the old instructions as a reference, they weren't right.

@sophiebits
Copy link
Collaborator

Agree. loose-envify doesn't matter either way for our built bundles but upgrading those four makes sense to me.

@gaearon
Copy link
Collaborator

gaearon commented Sep 6, 2017

Did lockfile update get lost? I don't see it in the PR now.
Never mind, the last comment says so.

@flarnie flarnie merged commit 8e46540 into facebook:master Sep 6, 2017
flarnie added a commit that referenced this pull request Sep 6, 2017
**what is the change?:**
This just updated the error codes on the docs.

For my future self, recording the steps it took to get here:
- On master, run `npm run build -- --extract-errors`
- Create a commit with those changes, and open a PR
  (#10619)
- Cherry-pick that commit onto 15-stable, and open a PR
  (#10621)
- Update the error codes on the `gh-pages branch` by doing the
  following based on docs
  (https://github.com/facebook/react/tree/master/docs#updating-facebookgithubioreact):
  - Set up a sister directory to `react` with `gh-pages` checked out,
    and call it `react-gh-pages`
  - In your `react` repo, with `15-stable` checked out and the
    error-codes update cherry-picked, run:
    `cd docs && bundle rake exec && bundle exec rake fetch_remotes && bundle exec rake release`
  - `cd ../react-gh-pages` and check in those changes, create a
    commit/PR
  - Manually test running the docs

**why make this change?:**
Even though this is just an RC, and not a final major release, we still
want the docs to be as up-to-date as possible.

**test plan:**
Run the docs locally and make sure things work.
(Flarnie will insert screenshots.)

**issue:**
#8854
@flarnie flarnie mentioned this pull request Sep 6, 2017
55 tasks
flarnie added a commit to flarnie/react that referenced this pull request Sep 8, 2017
* Update results.json

**what is the change?:**
Ran the build after updating Yarn lockfile and recorded latest build
size results.

**why make this change?:**
Let's get an idea of bundle size at this point, when we are preparing
the 16.0 RC.

**test plan:**
NA

**issue:**
facebook#8854

* Update error codes

https://github.com/facebook/react/tree/master/scripts/release-manager#update-the-error-codes
@flarnie flarnie deleted the updateReactFor16RC branch May 25, 2018 17:51
web-developer77 pushed a commit to web-developer77/facebook-react that referenced this pull request Jun 7, 2021
**what is the change?:**
This just updated the error codes on the docs.

For my future self, recording the steps it took to get here:
- On master, run `npm run build -- --extract-errors`
- Create a commit with those changes, and open a PR
  (facebook/react#10619)
- Cherry-pick that commit onto 15-stable, and open a PR
  (facebook/react#10621)
- Update the error codes on the `gh-pages branch` by doing the
  following based on docs
  (https://github.com/facebook/react/tree/master/docs#updating-facebookgithubioreact):
  - Set up a sister directory to `react` with `gh-pages` checked out,
    and call it `react-gh-pages`
  - In your `react` repo, with `15-stable` checked out and the
    error-codes update cherry-picked, run:
    `cd docs && bundle rake exec && bundle exec rake fetch_remotes && bundle exec rake release`
  - `cd ../react-gh-pages` and check in those changes, create a
    commit/PR
  - Manually test running the docs

**why make this change?:**
Even though this is just an RC, and not a final major release, we still
want the docs to be as up-to-date as possible.

**test plan:**
Run the docs locally and make sure things work.
(Flarnie will insert screenshots.)

**issue:**
facebook/react#8854
jerrydev0927 added a commit to jerrydev0927/react that referenced this pull request Jan 5, 2024
**what is the change?:**
This just updated the error codes on the docs.

For my future self, recording the steps it took to get here:
- On master, run `npm run build -- --extract-errors`
- Create a commit with those changes, and open a PR
  (facebook/react#10619)
- Cherry-pick that commit onto 15-stable, and open a PR
  (facebook/react#10621)
- Update the error codes on the `gh-pages branch` by doing the
  following based on docs
  (https://github.com/facebook/react/tree/master/docs#updating-facebookgithubioreact):
  - Set up a sister directory to `react` with `gh-pages` checked out,
    and call it `react-gh-pages`
  - In your `react` repo, with `15-stable` checked out and the
    error-codes update cherry-picked, run:
    `cd docs && bundle rake exec && bundle exec rake fetch_remotes && bundle exec rake release`
  - `cd ../react-gh-pages` and check in those changes, create a
    commit/PR
  - Manually test running the docs

**why make this change?:**
Even though this is just an RC, and not a final major release, we still
want the docs to be as up-to-date as possible.

**test plan:**
Run the docs locally and make sure things work.
(Flarnie will insert screenshots.)

**issue:**
facebook/react#8854
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants