Skip to content

Assorted fixes#27408

Merged
XhmikosR merged 8 commits intov4-devfrom
v4-dev-xmr-various
Oct 14, 2018
Merged

Assorted fixes#27408
XhmikosR merged 8 commits intov4-devfrom
v4-dev-xmr-various

Conversation

@XhmikosR
Copy link
Copy Markdown
Member

The popper.js situation is something we need to find a solution for, because we will definitely hit it again... We need a way to be in sync.

@XhmikosR XhmikosR requested review from Johann-S and mdo October 13, 2018 09:24
@XhmikosR XhmikosR force-pushed the v4-dev-xmr-various branch 2 times, most recently from 76f65a6 to 4da997e Compare October 13, 2018 09:46
@XhmikosR
Copy link
Copy Markdown
Member Author

@Johann-S: shouldn't we have a test which we expect to fail with invalid selectors?

@XhmikosR XhmikosR force-pushed the v4-dev-xmr-various branch 2 times, most recently from 6d46bf8 to 73177ce Compare October 13, 2018 10:26
@Johann-S
Copy link
Copy Markdown
Member

@XhmikosR
Copy link
Copy Markdown
Member Author

@Johann-S: yes, but shouldn't this throw and thus that test fail?

@Johann-S
Copy link
Copy Markdown
Member

Yep but we don't test our documentation in our unit test

@XhmikosR
Copy link
Copy Markdown
Member Author

That's not in the docs though, that's in tests.

@XhmikosR XhmikosR force-pushed the v4-dev-xmr-various branch 3 times, most recently from 3e5a0d2 to 3943f00 Compare October 13, 2018 13:35
@XhmikosR XhmikosR merged commit bcbf3f2 into v4-dev Oct 14, 2018
@XhmikosR XhmikosR deleted the v4-dev-xmr-various branch October 14, 2018 05:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants