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

Public Beta Testing of [email protected] #2172

Closed
gaearon opened this issue May 16, 2017 · 67 comments
Closed

Public Beta Testing of [email protected] #2172

gaearon opened this issue May 16, 2017 · 67 comments

Comments

@gaearon
Copy link
Contributor

gaearon commented May 16, 2017


Known Issues:

  • None (please report!)

Updates:

  • Published the first canary.
  • Published another canary that fixes the path issue and disables require.ensure() warning.
  • Published another canary that re-enables parsing of require.ensure().
  • Published a canary with more legible colors on Windows and port detection fixes
  • Published a canary with Jest fixes, stricter module resolution, Uglify fix, and more consistent overlay styling
  • Published a canary with fixed npm package, more visual fixes to overlays, better build messages, and smaller post eject code size
  • Final call for testing: Published the last canary with IE fixes.

Today I invite you to help us beta test [email protected] for a few days before the public release.
It used to be called 0.10 but we figured there’s enough improvements to make it 1.0.

To create a new app with it, run:

create-react-app my-app --scripts-version react-scripts@canary

To upgrade an existing app, replace "react-scripts": "0.9.5" in your package.json with:

    "react-scripts": "canary"

and run npm install (or yarn).

There is no changelog yet, but you can open the milestone to get a sense of included changes.

A few highlights:

There are a few more changes we plan to get into 1.0.0, but this should be enough the first public testing.

Please report any issues you bump into in this thread. We want this release to be as solid as possible, even though it’s unlikely we’ll keep it unreleased for much longer.

Big thanks to all the contributors. This will be our biggest release yet.

This was referenced May 16, 2017
@gaearon
Copy link
Contributor Author

gaearon commented May 18, 2017

Looks like some it could be a Yarn bug? cc @bestander

[email protected] asks for

"react-dev-utils": "^1.0.0-alpha.c82c4f05",

but I see

installed instead.

Try running npm install [email protected] as temporary fix.

@Timer
Copy link
Contributor

Timer commented May 18, 2017

@gaearon could it be the ^, where e > c? Maybe we should pass the --exact flag to lerna.

Yikes, looks like semver just functions like this:

@cdaringe
Copy link

cdaringe commented May 18, 2017

per the semver spec:

Precedence for two pre-release versions with the same major, minor, and patch version MUST be determined by comparing each dot separated identifier from left to right until a difference is found as follows: identifiers consisting of only digits are compared numerically and identifiers with letters or hyphens are compared lexically in ASCII sort order. Numeric identifiers always have lower precedence than non-numeric identifiers. A larger set of pre-release fields has a higher precedence than a smaller set, if all of the preceding identifiers are equal. Example: 1.0.0-alpha < 1.0.0-alpha.1 < 1.0.0-alpha.beta < 1.0.0-beta < 1.0.0-beta.2 < 1.0.0-beta.11 < 1.0.0-rc.1 < 1.0.0.

therefore, e280254d is > c82c4f05, which makes some amount of sense. the hash is still treated like a version number, so long as everything preceding is a match.

@gaearon
Copy link
Contributor Author

gaearon commented May 18, 2017

Huh. I guess Lerna --canary feature is a bit broken then.

@gaearon
Copy link
Contributor Author

gaearon commented May 18, 2017

Yep, it’s completely broken. 😄
lerna/lerna#277

I wish I knew that before.

@gaearon
Copy link
Contributor Author

gaearon commented May 18, 2017

Published a new canary with fixed npm package, more visual fixes to overlays, better build messages, and smaller post eject code size.

Please give it a try! It’s close to a release candidate.

@mxstbr
Copy link
Contributor

mxstbr commented May 18, 2017

Been running a custom fork of (what used to be) 0.10 in production for a month or two now (ever since the webpack 2 PR) and haven't run into any bugs/issues yet!

@gaearon
Copy link
Contributor Author

gaearon commented May 18, 2017

Just published an RC as the last canary.
Speak now or hold your peace 😄

@notrab
Copy link

notrab commented May 18, 2017

Just ran an upgrade and everything went smoothly. Just deployed to production too, everything running fine 🤘

@kentcdodds
Copy link
Contributor

The testing-workshop tests all passed without a hitch! And the app still looks perfect (both dev and prod versions)! 🚀 🚢

@RohovDmytro
Copy link

New canary caught me on some none-existed imported values. Fixing that led me to successfull deployment to prod!

@RohovDmytro
Copy link

RohovDmytro commented May 18, 2017

Feedback.

./src/components/PageResume.js
  Line 530:  'Wow' is not defined  react/jsx-no-undef

Search for the rule keywords to learn more about each error.

«rule keywords» was confusing for me for 15 seconds.

Word «rule» is confusing. What rule? I am not aware of any rules. This project hides config complexity, so a novice may not get what this message about.

We can make «react/jsx-no-undef» clickable , but maybe it's to much 😺

@TheRobBrennan
Copy link

@gaearon This is awesome!!! Thank you for adding that. This is absolutely beautiful now:
package_json_-_create-react-app-with-canary

@gaearon
Copy link
Contributor Author

gaearon commented May 18, 2017

@rogovdm I agree, changed to just keywords. Ideally I’d like to make them clickable but I don’t want to force people to use Google.

@mbifulco
Copy link

Updating an existing project - this is working great for me so far. Thanks to you and your team, @gaearon!

@ffxsam
Copy link

ffxsam commented May 19, 2017

I get get a new warning:
[eslint] Definition for rule 'jsx-a11y/alt-text' was not found (jsx-a11y/alt-text)

I get this warning now too, and can't figure out how to get rid of it. It's not documented in the README, and Googling around for this didn't turn up anything useful.

@gaearon
Copy link
Contributor Author

gaearon commented May 19, 2017

Where do you get it? You have not specified your editor, plugin, whether you use global ESLint for it, etc.

@gaearon
Copy link
Contributor Author

gaearon commented May 19, 2017

I'll close this issue since 1.0 is out but please file new ones if something is confusing!

@gaearon gaearon closed this as completed May 19, 2017
@ffxsam
Copy link

ffxsam commented May 19, 2017

Nuclide, latest version (of Atom as well). No global eslint installed.
BTW, I found the plugin & its documentation. Fixed it with the following in my .eslintrc:

  "rules": {
    "jsx-a11y/alt-text": "off",

@gaearon
Copy link
Contributor Author

gaearon commented May 19, 2017

That's not the right fix, you're still running wrong versions of plugins. This silences this warning but you'll see more breakage in the future. I suggest fixing the root cause rather than symptoms.

Let's try this:

  1. Ensure that in Atom ESLint settings you have "Use Global ESLint" turned off
  2. Run npm install in your project (or yarn)
  3. Quit Atom (ensure its process doesn't hang around)
  4. Start Atom again

I think that should help but let me know if not, and we can debug more.

@ffxsam
Copy link

ffxsam commented May 19, 2017

That did the trick! I noticed that .eslintrc changes would get recognized by closing a file and reopening it, but in this case, quitting Atom entirely was required.

@gaearon
Copy link
Contributor Author

gaearon commented May 19, 2017

Yea. Since it loads rules into memory, their code sticks around in the require cache even after closing the file.

@slightly-askew
Copy link

slightly-askew commented May 22, 2017

Hi gaearon & all, I'm getting the following flow error in both Nuclide and VS Code. I've tried the above steps you recommended several times but no joy. Also tried deleting node_modules and reinstalling.

identifier ESLintContext Could not resolve name

screen shot 2017-05-22 at 11 20 44 am

Doesn't happen in React Scripts v0.9.x; only when upgrading to 1.0.2-3

In Nuclide 0.228.0:

  • Linter is disabled
  • linter-eslint 8.1.7 is enabled with use global off

In VS Code 1.12.2:

  • ESLint is enabled

Here are my .eslintrc and .flowconfig settings:

screen shot 2017-05-22 at 11 01 41 am

devDependancies:

"devDependencies": {
    "@kadira/storybook": "^2.21.0",
    "eslint-plugin-jsx-a11y": "^5.0.3",
    "flow-bin": "^0.42.0",
    "react-scripts": "^1.0.3",
    "stylelint": "^7.10.1",
    "stylelint-config-standard": "^16.0.0"
  },

Any suggestions much appreciated!

@gaearon
Copy link
Contributor Author

gaearon commented May 22, 2017

The error is because Flow is trying to typecheck the ESLint plugin.
You probably want to add .*/eslint-plugin-jsx-a11y/.* to [ignore] in your .flowconfig.

@slightly-askew
Copy link

Ahh ok that worked, thanks very much :)

@zerubeus
Copy link

zerubeus commented Jun 13, 2017

@gaearon I don't have global ESlint, and I'm using VSCode, and I have this warning in both when working locally and the Tarvic CI is breaking because of this same warning : Definition for rule 'jsx-a11y/alt-text' was not found jsx-a11y/alt-text, this occurred after updating to react-script 1.0.7

@gaearon
Copy link
Contributor Author

gaearon commented Jun 13, 2017

@zerubeus The public testing has finished a long time ago so this is not an appropriate issue. Please file a new issue with an example project reproducing this. Thanks!

@facebook facebook locked and limited conversation to collaborators Jun 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests