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

Babel v7, ESlint fixes, React/React Navigation upgrades, mobx removed #3142

Merged

Conversation

daviscabral
Copy link
Collaborator

@daviscabral daviscabral commented Jul 23, 2018

This PR is just to follow the changes I am doing - while airbnb/javascript repo is not fully on Babel v7 - I had to build it locally (I really hate monorepos for situations like this).

supremebeing7 and others added 3 commits July 22, 2018 16:31
Copy link
Owner

@aksonov aksonov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK so far. Do you know how to solve error "/home/circleci/airbnb-style/packages/eslint-config-airbnb" doesn't exist. error?

@daviscabral
Copy link
Collaborator Author

@aksonov that's because I am using a local copy of eslint-config-airbnb. I'll push one temporary fix for it - while they do not release it.

@aksonov
Copy link
Owner

aksonov commented Jul 23, 2018

I guess scenes.test.js should be disabled now because it works with internal state...

@daviscabral
Copy link
Collaborator Author

@aksonov just fixed it - it was a problem with babel and jest.

@daviscabral
Copy link
Collaborator Author

Cool - only linter errors now.

@aksonov
Copy link
Owner

aksonov commented Jul 23, 2018

Btw, what about removing dist and use src? react-navigation doesn't do compilation now, right?

@daviscabral
Copy link
Collaborator Author

Done - using src now 👍.

}

if (tabs) {
let createTabNavigator = createMaterialTopTabNavigator;
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TabBarTop and TabBarBottom are deprecated.

@daviscabral
Copy link
Collaborator Author

I did already - but that limited a bit the way it can be customized.

@aksonov
Copy link
Owner

aksonov commented Jul 23, 2018

Yes, just found it. Don't worry, we will think about it later.

@aksonov
Copy link
Owner

aksonov commented Jul 23, 2018 via email

@daviscabral
Copy link
Collaborator Author

Feel free to merge it - I got a few other things to keep going and I'll need to postpone the rest of changes for later tonight or tomorrow morning.

@aksonov
Copy link
Owner

aksonov commented Jul 23, 2018 via email

@daviscabral daviscabral changed the title [DO NOT MERGE] Babel v7, ESlint fixes, React/React Navigation upgrades, mobx removed Babel v7, ESlint fixes, React/React Navigation upgrades, mobx removed Jul 24, 2018
@daviscabral
Copy link
Collaborator Author

daviscabral commented Jul 24, 2018

@aksonov initial work done.

You will need to clone airbnb/javascript in order to have it working while they don't release a babel v7 version of their presets/configs.

@aksonov
Copy link
Owner

aksonov commented Jul 24, 2018

Well done, merging!

@aksonov aksonov merged commit 49b8991 into aksonov:react-navigation-v2 Jul 24, 2018
@daviscabral daviscabral deleted the dc/rn-2-babel-example-eslint branch July 26, 2018 07:04
daviscabral added a commit to daviscabral/react-native-router-flux that referenced this pull request Jul 26, 2018
…e-action

* react-navigation-v2:
  small tweaks
  bump version
  introduce onStateChange handler to be used instead of createReducer
  fix navTransparent
  add onEnter/onExit handlers and automatic transitions
  fix eslint
  fix error
  more eslint fixes
  fix eslint errors
  fix error
  fix eslint
  fix custom navigators
  Babel v7, ESlint fixes, React/React Navigation upgrades, mobx removed (aksonov#3142)
  use WIP RNRF for Example
  revert RN to 0.55.4
  upgrade Example to RN0.56
  WIP: migration to react-navigation v2.x (aksonov#3068)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants