-
Notifications
You must be signed in to change notification settings - Fork 78
[Dev] Provide advice on how to clone, and provide a checklist for signing #558
Conversation
Doing many peeps so I can see if it handles that correct |
Huh, the formatting broke in this one? :o edit: it seems to be adding spaces after the |
Thanks - fixed, being redeployed atm |
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. |
Aww, it did. Hrm. |
❓ 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. |
2afc9d5
to
283763a
Compare
👍 |
@orta not sure why but 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 |
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 |
In theory it can be:
|
1b077a4
to
193ab65
Compare
Very small thing but |
Otherwise the setup instructions work for me! |
…t of assignees and some metadata for cloning
Good catch - I have enough open PRs, so self-merging this 👍 |
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:
Then run
xcrun simctl launch booted net.artsy.Emission
once a the simulator has finished bootingTo 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.