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

FIX: moves back padStart/padEnd to core polyfills #8714

Merged
merged 1 commit into from Jan 14, 2020

Conversation

@jjaffeux
Copy link
Contributor

jjaffeux commented Jan 14, 2020

Multiple users are on chrome 56 when this is only supported on chrome 57. Given it's only few lines of code, it makes sense to keep supporting this.

Multiple users are on chrome 56 when this is only supported on chrome 57. Given it's only few lines of code, it makes sense to keep supporting this.
@discoursebot

This comment has been minimized.

Copy link

discoursebot commented Jan 14, 2020

You've signed the CLA, jjaffeux. Thank you! This pull request is ready for review.

@jjaffeux jjaffeux merged commit f10078e into discourse:master Jan 14, 2020
1 check was pending
1 check was pending
DeepCode DeepCode is checking for issues.
@jjaffeux jjaffeux deleted the jjaffeux:pad-start-end-core branch Jan 14, 2020
@deepcode-ci-bot

This comment has been minimized.

Copy link

deepcode-ci-bot bot commented Jan 14, 2020

DeepCode's analysis on #86a6db found:

  • 0 critical issues. ⚠️ 0 warnings and 0 minor issues. ✔️ 0 issues were fixed.

💬 This comment has been generated by the DeepCode bot, installed by the owner of the repository. The DeepCode bot protects your repository by detecting and commenting on security vulnerabilities or other critical issues.


☺️ If you want to provide feedback on our bot, here is how to contact us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.