Skip to content

Commit

Permalink
Prettier
Browse files Browse the repository at this point in the history
  • Loading branch information
acdlite committed Jan 21, 2022
1 parent 26baf6a commit ca143e1
Showing 1 changed file with 2 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -213,10 +213,7 @@ function renameStyle(
instance.setNativeProps({style: newStyle});
} else if (isArray(style)) {
const lastIndex = style.length - 1;
if (
typeof style[lastIndex] === 'object' &&
!isArray(style[lastIndex])
) {
if (typeof style[lastIndex] === 'object' && !isArray(style[lastIndex])) {
customStyle = shallowClone(style[lastIndex]);
delete customStyle[oldName];
if (newName) {
Expand Down Expand Up @@ -299,10 +296,7 @@ function setStyle(
instance.setNativeProps({style: newStyle});
} else if (isArray(style)) {
const lastLength = style.length - 1;
if (
typeof style[lastLength] === 'object' &&
!isArray(style[lastLength])
) {
if (typeof style[lastLength] === 'object' && !isArray(style[lastLength])) {
agent.overrideValueAtPath({
type: 'props',
id,
Expand Down

6 comments on commit ca143e1

@bvaughn
Copy link
Contributor

Choose a reason for hiding this comment

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

Ooof. #23154 had no CI failures. Looking again, it looks like most of the jobs didn't run and I just didn't notice that.

@acdlite
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

😮

@gaearon
Copy link
Collaborator

Choose a reason for hiding this comment

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

do you guys have a time machine

Screenshot 2022-01-21 at 16 04 37

@bvaughn
Copy link
Contributor

Choose a reason for hiding this comment

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

We enjoyed the interaction so much we had to do it again.

@behnammodi
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, this was my mistake.

@bvaughn
Copy link
Contributor

Choose a reason for hiding this comment

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

No worries @behnammodi. Our CI should have caught this on the PR.

Please sign in to comment.