Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

[Dev] Provide advice on how to clone, and provide a checklist for signing #558

Merged
merged 2 commits into from
May 27, 2017

Conversation

orta
Copy link
Contributor

@orta orta commented May 22, 2017

This is WIP - need to null out whether the tick is checked in the "has this been changed" and thus needs to update.


Tested on Device?

How to get set up with this PR?

 

To run on your computer:

git fetch origin pull/558/head:orta-558-checkout
git checkout orta-558-checkout
yarn install
cd example; pod install; cd ..
open -a Simulator
yarn start

Then run xcrun simctl launch booted net.artsy.Emission once a the simulator has finished booting

To run inside Eigen (prod or beta) or Emission (beta): Shake the phone to get the Admin menu.

If you see "Use Staging React Env" - click that and restart, then follow the next step.

Click on "Choose an RN build" - then pick the one that says: "X,Y,Z"

Note: this is a TODO for PRs, currently you can only do it on master commits.

@orta
Copy link
Contributor Author

orta commented May 22, 2017

Doing many peeps so I can see if it handles that correct

@mennenia
Copy link
Contributor

mennenia commented May 23, 2017

Huh, the formatting broke in this one? :o

edit: it seems to be adding spaces after the < tags

@orta
Copy link
Contributor Author

orta commented May 23, 2017

Thanks - fixed, being redeployed atm

@orta orta changed the title [Dev][WIP] Provide advice on how to clone, and provide a checklist for signing [Dev] Provide advice on how to clone, and provide a checklist for signing May 23, 2017
@orta
Copy link
Contributor Author

orta commented May 23, 2017

OK, it'll now it won't reset the ticks on new danger runs. I think, we'll find out if mine + @mennenia's ticks disappear.

@orta
Copy link
Contributor Author

orta commented May 23, 2017

Aww, it did. Hrm.

@orta
Copy link
Contributor Author

orta commented May 23, 2017

❓ Anyone got an idea, this looks like I'd expect.

> const a = "- [x] asdasdasdda;\n - [ ] asdazzxxzxc".replace(/- \[*.\]/g, "---")
undefined
> const b = "- [x] asdasdasdda;\n - [x] asdazzxxzxc".replace(/- \[*.\]/g, "---")
undefined
> a === b
true

Wonder if there's an extra newline somewhere.

@orta orta force-pushed the danger-edit-pr branch 2 times, most recently from 2afc9d5 to 283763a Compare May 23, 2017 09:25
@orta
Copy link
Contributor Author

orta commented May 23, 2017

It looks like it was whitespacing

screen shot 2017-05-23 at 10 27 26

@orta
Copy link
Contributor Author

orta commented May 23, 2017

👍

@l2succes
Copy link
Contributor

l2succes commented May 23, 2017

screen shot 2017-05-23 at 12 20 37 pm

@orta not sure why but yarn ios doesn't work for me so I have to run Xcode manually. Not a blocker but making a note in case it happens to other people.

Edit: I saw this issue in the react-native repo, tried a few things, nothing worked. I'll report back when I find a fix facebook/react-native#11293

@orta
Copy link
Contributor Author

orta commented May 23, 2017

To my knowledge it doesn't do much - we might be able to replicate it from our side pretty easily, we'd need a command that opens the right sim, then with the right app

@orta
Copy link
Contributor Author

orta commented May 23, 2017

In theory it can be:

open -a Simulator
echo "Once the simulator has loaded: xcrun simctl launch booted net.artsy.Emission"

@sarahscott
Copy link
Contributor

Very small thing but cd example; pod install cd .. should be cd example; pod install; cd ..

@sarahscott
Copy link
Contributor

Otherwise the setup instructions work for me!

…t of assignees and some metadata for cloning
@orta
Copy link
Contributor Author

orta commented May 27, 2017

Good catch - I have enough open PRs, so self-merging this 👍

@orta orta merged commit db885cb into master May 27, 2017
@orta orta deleted the danger-edit-pr branch May 27, 2017 13:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants